[GitHub] [camel-k] astefanutti commented on a diff in pull request #4120: fix(knative): Integration in error state when knative SinkBinding mode=inclusion

[via GitHub]

astefanutti commented on code in PR #4120:
URL: https://github.com/apache/camel-k/pull/4120#discussion_r1133947864


##########
config/rbac/operator-role.yaml:
##########
@@ -80,6 +80,13 @@ rules:
   - patch
   - update
   - watch
+- apiGroups:

Review Comment:
   I see what you mean. Then having a new role with an explicit name, so it's 
clear it's not part of the core / mandatory roles, would work even better IMO.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org