put mutex around installplan creation

Signed-off-by: akihikokuroda <akihikokuroda2020@gmail.com>
operator-framework · Jan 5, 2022 · f29ae6d · f29ae6d
1 parent 20a4381
commit f29ae6d
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pkg/controller/operators/catalog/operator.go b/pkg/controller/operators/catalog/operator.go
@@ -116,6 +116,7 @@ type Operator struct {
  installPlanTimeout time.Duration
  bundleUnpackTimeout time.Duration
  clientFactory clients.Factory
+ muInstallPlan sync.Mutex
 }
 
 type CatalogSourceSyncFunc func(logger *logrus.Entry, in *v1alpha1.CatalogSource) (out *v1alpha1.CatalogSource, continueSync bool, syncError error)
@@ -1162,6 +1163,9 @@ func (o *Operator) ensureInstallPlan(logger *logrus.Entry, namespace string, gen
  return nil, err
  }
 
+ o.muInstallPlan.Lock()
+ defer o.muInstallPlan.Unlock()
+
  for _, installPlan := range installPlans {
  if installPlan.Spec.Generation == gen {
  return reference.GetReference(installPlan)