Merge pull request #7174 from reasonerjt/snapshot-flag-skip-csi

Make sure the PVs skipped by CSI plugin due to settings in backup spec are tracked
This commit is contained in:
Xun Jiang/Bruce Jiang
2023-12-05 09:31:21 +08:00
committed by GitHub
+2 -2
View File
@@ -384,8 +384,8 @@ func (ib *itemBackupper) executeActions(
// snapshot was skipped by CSI plugin
ib.trackSkippedPV(obj, groupResource, csiSnapshotApproach, "skipped b/c it's not a CSI volume", log)
delete(u.GetAnnotations(), skippedNoCSIPVAnnotation)
} else if actionName == csiBIAPluginName || actionName == vsphereBIAPluginName {
// the snapshot has been taken
} else if (actionName == csiBIAPluginName || actionName == vsphereBIAPluginName) && !boolptr.IsSetToFalse(ib.backupRequest.Backup.Spec.SnapshotVolumes) {
// the snapshot has been taken by the BIA plugin
ib.unTrackSkippedPV(obj, groupResource, log)
}
mustInclude := u.GetAnnotations()[mustIncludeAdditionalItemAnnotation] == "true" || finalize