Skip to content

Commit 644b414

Browse files
jparrillclaude
andcommitted
fix(remove-datamover): remove ValidateDataMover and snapshot polling logic
Remove the plugin-side snapshot/datamover polling since Velero already handles VolumeSnapshotContent, VolumeSnapshot, DataUpload, and PodVolumeBackup lifecycle natively. This eliminates duplicated work and simplifies the plugin significantly. Removed: - ValidateDataMover, reconcileStandardDataMover, reconcileAzureDataMover - All Check/Wait/Reconcile functions for snapshots and data uploads - BlackList struct and all associated functions - PodVolumeBackup polling logic - DataUploadTimeout/DataUploadCheckPace configuration options - All associated tests (~5000 lines removed) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Juan Manuel Parrilla Madrid <jparrill@redhat.com>
1 parent caa3bfa commit 644b414

8 files changed

Lines changed: 11 additions & 5021 deletions

File tree

pkg/common/types.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
package common
22

3-
import "time"
4-
53
const (
64
CommonBackupAnnotationName string = "hypershift.openshift.io/common-backup-plugin"
75
CommonRestoreAnnotationName string = "hypershift.openshift.io/common-restore-plugin"
@@ -13,10 +11,6 @@ const (
1311

1412
PluginConfigMapName string = "hypershift-oadp-plugin-config"
1513

16-
// Default values for the backup plugin.
17-
defaultPVBackupTimeout time.Duration = 30 * time.Minute
18-
defaultPVBackupCheckPace time.Duration = 10 * time.Second
19-
2014
DefaultK8sSAFilePath string = "/var/run/secrets/kubernetes.io/serviceaccount"
2115
KubevirtRHCOSLabel string = "hypershift.openshift.io/is-kubevirt-rhcos"
2216

0 commit comments

Comments
 (0)