From 67535449498943ac338a2fa22d097a323f592de8 Mon Sep 17 00:00:00 2001 From: Janet Kuo Date: Fri, 17 Feb 2017 11:29:39 -0800 Subject: [PATCH] Fix cronjob panic on status update failure --- pkg/controller/cronjob/cronjob_controller.go | 1 + 1 file changed, 1 insertion(+) diff --git a/pkg/controller/cronjob/cronjob_controller.go b/pkg/controller/cronjob/cronjob_controller.go index b8b778d5e55..173e2a25d31 100644 --- 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