Fix cronjob panic on status update failure

lavalamp · Feb 17, 2017 · 6753544 · 6753544
1 parent 68814c0
commit 6753544
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/pkg/controller/cronjob/cronjob_controller.go b/pkg/controller/cronjob/cronjob_controller.go
@@ -167,6 +167,7 @@ func SyncOne(sj batch.CronJob, js []batch.Job, now time.Time, jc jobControlInter
 	updatedSJ, err := sjc.UpdateStatus(&sj)
 	if err != nil {
 		glog.Errorf("Unable to update status for %s (rv = %s): %v", nameForLog, sj.ResourceVersion, err)
+		return
 	}
 	sj = *updatedSJ