diff --git a/internal/controller/velero.go b/internal/controller/velero.go index fe8fee1a12..3bf2eb0551 100644 --- a/internal/controller/velero.go +++ b/internal/controller/velero.go @@ -679,6 +679,14 @@ func (r *DataProtectionApplicationReconciler) customizeVeleroContainer(veleroCon if err != nil { return err } + // Re-add uploader-type since GetArgs() doesn't include it + // but it comes from NodeAgent configuration, not Velero.Args + if dpa.Spec.Configuration.NodeAgent != nil && + len(dpa.Spec.Configuration.NodeAgent.UploaderType) > 0 { + uploaderType := dpa.Spec.Configuration.NodeAgent.UploaderType + veleroContainer.Args = append(veleroContainer.Args, + fmt.Sprintf("--uploader-type=%s", uploaderType)) + } } return nil } diff --git a/internal/controller/velero_test.go b/internal/controller/velero_test.go index 03b7e49141..945cdb12a0 100644 --- a/internal/controller/velero_test.go +++ b/internal/controller/velero_test.go @@ -2350,11 +2350,11 @@ func TestDPAReconciler_buildVeleroDeployment(t *testing.T) { veleroDeployment: testVeleroDeployment.DeepCopy(), wantVeleroDeployment: createTestBuiltVeleroDeployment(TestBuiltVeleroDeploymentOptions{ args: []string{ - // should be present... "--uploader-type=kopia", "--client-burst=321", "--client-qps=321", "--fs-backup-timeout=4h0m0s", defaultRestoreResourcePriorities, + "--uploader-type=kopia", defaultDisableInformerCache, }, }),