From 5df275539968239abb02db4b72b2ff8586c615f6 Mon Sep 17 00:00:00 2001 From: Maciej Szulik Date: Thu, 10 May 2018 11:20:22 +0200 Subject: [PATCH] Never clean backoff in job controller --- pkg/controller/job/job_controller.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/controller/job/job_controller.go b/pkg/controller/job/job_controller.go index 3d25623df98..dc7688f2e12 100644 --- a/pkg/controller/job/job_controller.go +++ b/pkg/controller/job/job_controller.go @@ -356,10 +356,10 @@ func (jm *JobController) enqueueController(obj interface{}, immediate bool) { return } - if immediate { - jm.queue.Forget(key) + backoff := time.Duration(0) + if !immediate { + backoff = getBackoff(jm.queue, key) } - backoff := getBackoff(jm.queue, key) // TODO: Handle overlapping controllers better. Either disallow them at admission time or // deterministically avoid syncing controllers that fight over pods. Currently, we only