Merge pull request #1939 from awgreene/fix-operatorcondition-status-verbs

openshift-merge-robot · web-flow · commit abe648a8b0a1 · 2021-01-05T23:57:53.000-05:00
Bug 1905850: Fix operatorcondition role verbs
diff --git a/pkg/controller/operators/operatorcondition_controller.go b/pkg/controller/operators/operatorcondition_controller.go
@@ -115,7 +115,7 @@ func (r *OperatorConditionReconciler) ensureOperatorConditionRole(operatorCondit
 				ResourceNames: []string{operatorCondition.GetName()},
 			},
 			{
-				Verbs:         []string{"get,update,patch"},
+				Verbs:         []string{"get", "update", "patch"},
 				APIGroups:     []string{"operators.coreos.com"},
 				Resources:     []string{"operatorconditions/status"},
 				ResourceNames: []string{operatorCondition.GetName()},
diff --git a/pkg/controller/operators/operatorcondition_controller_test.go b/pkg/controller/operators/operatorcondition_controller_test.go
@@ -106,7 +106,7 @@ var _ = Describe("OperatorCondition", func() {
 						ResourceNames: []string{namespacedName.Name},
 					},
 					{
-						Verbs:         []string{"get,update,patch"},
+						Verbs:         []string{"get", "update", "patch"},
 						APIGroups:     []string{"operators.coreos.com"},
 						Resources:     []string{"operatorconditions/status"},
 						ResourceNames: []string{namespacedName.Name},

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ func (r *OperatorConditionReconciler) ensureOperatorConditionRole(operatorCondit`
`115`	`115`	`ResourceNames: []string{operatorCondition.GetName()},`
`116`	`116`	`},`
`117`	`117`	`{`
`118`		`- Verbs: []string{"get,update,patch"},`
	`118`	`+ Verbs: []string{"get", "update", "patch"},`
`119`	`119`	`APIGroups: []string{"operators.coreos.com"},`
`120`	`120`	`Resources: []string{"operatorconditions/status"},`
`121`	`121`	`ResourceNames: []string{operatorCondition.GetName()},`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ var _ = Describe("OperatorCondition", func() {`
`106`	`106`	`ResourceNames: []string{namespacedName.Name},`
`107`	`107`	`},`
`108`	`108`	`{`
`109`		`- Verbs: []string{"get,update,patch"},`
	`109`	`+ Verbs: []string{"get", "update", "patch"},`
`110`	`110`	`APIGroups: []string{"operators.coreos.com"},`
`111`	`111`	`Resources: []string{"operatorconditions/status"},`
`112`	`112`	`ResourceNames: []string{namespacedName.Name},`