SeedImage: reset download URL on Pod deletion

fgiudici · fgiudici · commit 19d94a808877 · 2023-05-26T15:54:10.000+02:00
Signed-off-by: Francesco Giudici &lt;francesco.giudici@suse.com&gt;
diff --git a/controllers/seedimage_controller.go b/controllers/seedimage_controller.go
@@ -458,6 +458,7 @@ func (r *SeedImageReconciler) deleteChildResources(ctx context.Context, seedImg
 			return err
 		}
 	} else {
+		// Pod found, delete it
 		if !util.IsObjectOwned(&foundPod.ObjectMeta, seedImg.UID) {
 			return fmt.Errorf("pod %s/%s doesn't belong to seedimage %s/%s", podNamespace, podName, seedImg.Namespace, seedImg.Name)
 		}
@@ -466,6 +467,7 @@ func (r *SeedImageReconciler) deleteChildResources(ctx context.Context, seedImg
 			return err
 		}
 	}
+	seedImg.Status.DownloadURL = ""
 
 	foundSvc := &corev1.Service{}
 	svcName := seedImg.Name

Original file line number	Diff line number	Diff line change
`@@ -458,6 +458,7 @@ func (r *SeedImageReconciler) deleteChildResources(ctx context.Context, seedImg`
`458`	`458`	`return err`
`459`	`459`	`}`
`460`	`460`	`} else {`
	`461`	`+ // Pod found, delete it`
`461`	`462`	`if !util.IsObjectOwned(&foundPod.ObjectMeta, seedImg.UID) {`
`462`	`463`	`return fmt.Errorf("pod %s/%s doesn't belong to seedimage %s/%s", podNamespace, podName, seedImg.Namespace, seedImg.Name)`
`463`	`464`	`}`
`@@ -466,6 +467,7 @@ func (r *SeedImageReconciler) deleteChildResources(ctx context.Context, seedImg`
`466`	`467`	`return err`
`467`	`468`	`}`
`468`	`469`	`}`
	`470`	`+ seedImg.Status.DownloadURL = ""`
`469`	`471`
`470`	`472`	`foundSvc := &corev1.Service{}`
`471`	`473`	`svcName := seedImg.Name`