Fix dry-run deletePods

artem-zinnatullin · lwolf · commit 9366e14b6f01 · 2020-06-04T11:15:02.000+01:00
diff --git a/pkg/controller/controller.go b/pkg/controller/controller.go
@@ -227,6 +227,7 @@ func (c *Kleaner) deleteJobs(job *batchv1.Job) {
 func (c *Kleaner) deletePods(pod *corev1.Pod) {
 	if c.dryRun {
 		log.Printf("dry-run: Pod '%s:%s' would have been deleted", pod.Namespace, pod.Name)
+		return
 	}
 	log.Printf("Deleting pod '%s:%s'", pod.Namespace, pod.Name)
 	var po metav1.DeleteOptions

Original file line number	Diff line number	Diff line change
`@@ -227,6 +227,7 @@ func (c Kleaner) deleteJobs(job batchv1.Job) {`
`227`	`227`	`func (c Kleaner) deletePods(pod corev1.Pod) {`
`228`	`228`	`if c.dryRun {`
`229`	`229`	`log.Printf("dry-run: Pod '%s:%s' would have been deleted", pod.Namespace, pod.Name)`
	`230`	`+ return`
`230`	`231`	`}`
`231`	`232`	`log.Printf("Deleting pod '%s:%s'", pod.Namespace, pod.Name)`
`232`	`233`	`var po metav1.DeleteOptions`