Skip to content

Commit da67b54

Browse files
committed
fix(installplan): fix race in installplan resolution
we shouldn't process an installplan unless it has had its status set by the subscription resolver
1 parent a6eafeb commit da67b54

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

pkg/controller/operators/catalog/operator.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -666,6 +666,12 @@ func (o *Operator) syncInstallPlans(obj interface{}) (syncError error) {
666666
})
667667

668668
logger.Info("syncing")
669+
670+
if len(plan.Status.Plan) == 0 {
671+
logger.Info("skip processing installplan without status - subscription sync responsible for initial status")
672+
return
673+
}
674+
669675
outInstallPlan, syncError := transitionInstallPlanState(logger.Logger, o, *plan)
670676

671677
if syncError != nil {

0 commit comments

Comments
 (0)