Skip to content

Commit 20118b4

Browse files
committed
Fix operatorcondition role verbs
Problem: The role created to allow operators to modify its OperatorCondition/status resource currently contains a string matching "get,patch,update" instead of an array of strings consisting of "get", "patch", and "update". Solution: Update the verbs defined for the role to match the expected array.
1 parent ad06749 commit 20118b4

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

pkg/controller/operators/operatorcondition_controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ func (r *OperatorConditionReconciler) ensureOperatorConditionRole(operatorCondit
115115
ResourceNames: []string{operatorCondition.GetName()},
116116
},
117117
{
118-
Verbs: []string{"get,update,patch"},
118+
Verbs: []string{"get", "update", "patch"},
119119
APIGroups: []string{"operators.coreos.com"},
120120
Resources: []string{"operatorconditions/status"},
121121
ResourceNames: []string{operatorCondition.GetName()},

pkg/controller/operators/operatorcondition_controller_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ var _ = Describe("OperatorCondition", func() {
106106
ResourceNames: []string{namespacedName.Name},
107107
},
108108
{
109-
Verbs: []string{"get,update,patch"},
109+
Verbs: []string{"get", "update", "patch"},
110110
APIGroups: []string{"operators.coreos.com"},
111111
Resources: []string{"operatorconditions/status"},
112112
ResourceNames: []string{namespacedName.Name},

0 commit comments

Comments
 (0)