Fix unpack job cache issue (#3204)

In the same vein as
operator-framework/operator-lifecycle-manager#3202,
use update if the unpack job already exists but isn't cached

Signed-off-by: kevinrizza <[email protected]>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 47aaa6be4a951c6bd8be016f6610c5319adc6a47

Author: kevinrizza

staging/operator-lifecycle-manager/pkg/controller/bundle/bundle_unpacker.go

@@ -668,6 +668,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
 	}
 
@@ -682,6 +685,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{})
+					}
 				}
 			}