Fix namespaceSelector logic

dhaiducek · openshift-merge-robot · commit 9c3f62fe6392 · 2022-10-06T21:02:03.000+02:00
Signed-off-by: Dale Haiducek &lt;dhaiduce@redhat.com&gt;
diff --git a/internal/plugin.go b/internal/plugin.go
@@ -599,8 +599,8 @@ func (p *Plugin) applyDefaults(unmarshaledConfig map[string]interface{}) {
 			}
 
 			selector := manifest.NamespaceSelector
-			if selector.Exclude != nil || selector.Include != nil ||
-				selector.MatchLabels != nil || selector.MatchExpressions != nil {
+			if selector.Exclude == nil && selector.Include == nil &&
+				selector.MatchLabels == nil && selector.MatchExpressions == nil {
 				manifest.NamespaceSelector = policy.NamespaceSelector
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -599,8 +599,8 @@ func (p *Plugin) applyDefaults(unmarshaledConfig map[string]interface{}) {`
`599`	`599`	`}`
`600`	`600`
`601`	`601`	`selector := manifest.NamespaceSelector`
`602`		`- if selector.Exclude != nil \|\| selector.Include != nil \|\|`
`603`		`- selector.MatchLabels != nil \|\| selector.MatchExpressions != nil {`
	`602`	`+ if selector.Exclude == nil && selector.Include == nil &&`
	`603`	`+ selector.MatchLabels == nil && selector.MatchExpressions == nil {`
`604`	`604`	`manifest.NamespaceSelector = policy.NamespaceSelector`
`605`	`605`	`}`
`606`	`606`