Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Always send VMBACKUP_EVENT_GENERIC_MANIFEST during quiesced snapshots.
vSphere 6.7 added a host-side interface that allows VMTools to send a "generic" backup manifest during a quiesced snapshot on Linux guests. VMTools 10.2.0 or later tries to notify the host of the backup manifest file through a vmbackup event message VMBACKUP_EVENT_GENERIC_MANIFEST. If the host is unable to field the message, then VMTools logs the failure and then continues with the quiesced snapshot in the older fashion, without the backup manifest. An earlier change attempted to reduce the amount of logging done when running on older hosts that don't support VMBACKUP_EVENT_GENERIC_MANIFEST by detecting when sending VMBACKUP_EVENT_GENERIC_MANIFEST fails and not sending the message again for subsequent quiesced snapshots. However, subsequent stress testing has uncovered problems with this approach when running on newer hosts; specifically, errors may sometimes be encountered on newer hosts when sending VMBACKUP_EVENT_GENERIC_MANIFEST. Therefore this change backs out that earlier change. Note that the need to solve the problem that that earlier change was intended to solve has been reduced because support for VMBACKUP_EVENT_GENERIC_MANIFEST has been backported to vSphere 6.5 P03, which is available, and vSphere 6.0 P08, which is scheduled for release later this year. ESXi 5.5 is out of general support. This change also addresses an issue that surfaced when testing on a host without support for VMBACKUP_EVENT_GENERIC_MANIFEST. If VMTools fails to send VMBACKUP_EVENT_GENERIC_MANIFEST, the quiesced snapshot operation will be aborted rather than continuing as it should. To address this, create a new function, VmBackup_SendEventNoAbort, which does not abort the quiesced snapshot on failure, and call that function rather than VmBackup_SendEvent when sending VMBACKUP_EVENT_GENERIC_MANIFEST.
- Loading branch information
1 parent
0c91747
commit c31710b
Showing
5 changed files
with
50 additions
and
82 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters