Fix unpack job cache issue (operator-framework#3204)

In the same vein as operator-framework#3202, use update if the unpack job already exists but isn't cached Signed-off-by: kevinrizza <krizza@redhat.com>
dtfranz · Apr 17, 2024 · 47aaa6b · 47aaa6b
1 parent a7b6658
commit 47aaa6b
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/pkg/controller/bundle/bundle_unpacker.go b/pkg/controller/bundle/bundle_unpacker.go
@@ -671,6 +671,9 @@ func (c *ConfigMapUnpacker) ensureJob(cmRef *corev1.ObjectReference, bundlePath
  }
  if len(jobs) == 0 {
  job, err = c.client.BatchV1().Jobs(fresh.GetNamespace()).Create(context.TODO(), fresh, metav1.CreateOptions{})
+ if apierrors.IsAlreadyExists(err) {
+ job, err = c.client.BatchV1().Jobs(fresh.GetNamespace()).Update(context.TODO(), fresh, metav1.UpdateOptions{})
+ }
  return
  }
 
@@ -685,6 +688,9 @@ func (c *ConfigMapUnpacker) ensureJob(cmRef *corev1.ObjectReference, bundlePath
  if time.Now().After(cond.LastTransitionTime.Time.Add(unpackRetryInterval)) {
  fresh.SetName(names.SimpleNameGenerator.GenerateName(fresh.GetName()))
  job, err = c.client.BatchV1().Jobs(fresh.GetNamespace()).Create(context.TODO(), fresh, metav1.CreateOptions{})
+ if apierrors.IsAlreadyExists(err) {
+ job, err = c.client.BatchV1().Jobs(fresh.GetNamespace()).Update(context.TODO(), fresh, metav1.UpdateOptions{})
+ }
  }
  }