fix: check latestMoverStatus.Result instead of it being nil

Signed-off-by: Devin Buhl <devin@buhl.casa>
backube · Jul 27, 2023 · da6cadf · da6cadf
1 parent d314308
commit da6cadf
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/controllers/mover/rclone/mover.go b/controllers/mover/rclone/mover.go
@@ -232,7 +232,7 @@ func (m *Mover) ensureJob(ctx context.Context, dataPVC *corev1.PersistentVolumeC
 		if m.paused {
 			parallelism = int32(0)
 		}
-		if !m.isImmediate && m.latestMoverStatus == nil {
+		if !m.isImmediate && m.latestMoverStatus.Result == "" {
 			parallelism = int32(0)
 		}
 		job.Spec.Parallelism = &parallelism

diff --git a/controllers/mover/restic/mover.go b/controllers/mover/restic/mover.go
@@ -294,7 +294,7 @@ func (m *Mover) ensureJob(ctx context.Context, cachePVC *corev1.PersistentVolume
 		if m.paused {
 			parallelism = int32(0)
 		}
-		if !m.isImmediate && m.latestMoverStatus == nil {
+		if !m.isImmediate && m.latestMoverStatus.Result == "" {
 			parallelism = int32(0)
 		}
 		job.Spec.Parallelism = &parallelism

diff --git a/controllers/mover/rsync/mover.go b/controllers/mover/rsync/mover.go
@@ -357,7 +357,7 @@ func (m *Mover) ensureJob(ctx context.Context, dataPVC *corev1.PersistentVolumeC
 		if m.paused {
 			parallelism = int32(0)
 		}
-		if !m.isImmediate && m.latestMoverStatus == nil {
+		if !m.isImmediate && m.latestMoverStatus.Result == "" {
 			parallelism = int32(0)
 		}
 		job.Spec.Parallelism = &parallelism

diff --git a/controllers/mover/rsynctls/mover.go b/controllers/mover/rsynctls/mover.go
@@ -364,7 +364,7 @@ func (m *Mover) ensureJob(ctx context.Context, dataPVC *corev1.PersistentVolumeC
 		if m.paused {
 			parallelism = int32(0)
 		}
-		if !m.isImmediate && m.latestMoverStatus == nil {
+		if !m.isImmediate && m.latestMoverStatus.Result == "" {
 			parallelism = int32(0)
 		}
 		job.Spec.Parallelism = &parallelism