Only check the owner of the service account if the service account has owner

horis233 · horis233 · commit 8990b2b9d244 · 2020-11-28T00:45:40.000-05:00
diff --git a/pkg/controller/operators/olm/requirements.go b/pkg/controller/operators/olm/requirements.go
@@ -273,7 +273,7 @@ func (a *Operator) permissionStatus(strategyDetailsDeployment *v1alpha1.Strategy
 			}
 
 			// Check if the ServiceAccount is owned by CSV
-			if !ownerutil.IsOwnedBy(sa, csv) {
+			if len(sa.GetOwnerReferences()) != 0 && !ownerutil.IsOwnedBy(sa, csv) {
 				met = false
 				status.Status = v1alpha1.RequirementStatusReasonPresentNotSatisfied
 				status.Message = "Service account is not owned by this ClusterServiceVersion"

Original file line number	Diff line number	Diff line change
`@@ -273,7 +273,7 @@ func (a Operator) permissionStatus(strategyDetailsDeployment v1alpha1.Strategy`
`273`	`273`	`}`
`274`	`274`
`275`	`275`	`// Check if the ServiceAccount is owned by CSV`
`276`		`- if !ownerutil.IsOwnedBy(sa, csv) {`
	`276`	`+ if len(sa.GetOwnerReferences()) != 0 && !ownerutil.IsOwnedBy(sa, csv) {`
`277`	`277`	`met = false`
`278`	`278`	`status.Status = v1alpha1.RequirementStatusReasonPresentNotSatisfied`
`279`	`279`	`status.Message = "Service account is not owned by this ClusterServiceVersion"`