New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug 1826716: RBAC to authorise service account to list CRDs #413
Bug 1826716: RBAC to authorise service account to list CRDs #413
Conversation
/assign @spadgett |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change itself looks good. Not suer if anything additional is needed in order to make it available to unprivileged user to list the CRDs. @benjaminapetersen thoughts ?
182e37d
to
e5c8953
Compare
/retest |
1 similar comment
/retest |
/approve /retest unrelated things i think. |
/retest |
6 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
e5c8953
to
eb484a0
Compare
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some additional feedback, regarding structure and naming.
eb484a0
to
1ed47ce
Compare
/retest |
@abhi-kn at this point we need a BZ for the ci-bot to merge the PR |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhi-kn, benjaminapetersen, jhadvig The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@abhi-kn: This pull request references Bugzilla bug 1826716, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest Please review the full test history for this PR and help us cut down flakes. |
4 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@abhi-kn: All pull requests linked via external trackers have merged: openshift/console-operator#413. Bugzilla bug 1826716 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
https://issues.redhat.com/browse/ODC-3398
New endpoint is being added in console backend to list knative event source CRDs.
Ref: openshift/console#4945.
In order to allow an unprivileged user to get this information without read access to CRDs, new RBAC has been added here to authorise service account to list CRDs.