Skip to content

Conversation

enj
Copy link
Contributor

@enj enj commented Nov 9, 2017

registry-admin can already manage these resources via the proxied origin authorization endpoints. This just allows it to perform these actions directly.

Signed-off-by: Monis Khan [email protected]

/assign @simo5 @deads2k

/kind bug

registry-admin can already manage these resources via the proxied
origin authorization endpoints.  This just allows it to perform
these actions directly.

Signed-off-by: Monis Khan <[email protected]>
@openshift-ci-robot openshift-ci-robot added kind/bug Categorizes issue or PR as related to a bug. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 9, 2017
@openshift-merge-robot openshift-merge-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 9, 2017
@enj
Copy link
Contributor Author

enj commented Nov 9, 2017

@openshift/sig-security not sure if this cluster role is used, but this was just an oversight during the RBAC migration.

@simo5
Copy link
Contributor

simo5 commented Nov 9, 2017

/lgtm
let's keep things consistent anyway, we can open an issue to review if these permissions are still needed

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 9, 2017
@openshift-merge-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: enj, simo5

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@enj
Copy link
Contributor Author

enj commented Nov 9, 2017

/retest

4 similar comments
@enj
Copy link
Contributor Author

enj commented Nov 9, 2017

/retest

@enj
Copy link
Contributor Author

enj commented Nov 9, 2017

/retest

@enj
Copy link
Contributor Author

enj commented Nov 9, 2017

/retest

@enj
Copy link
Contributor Author

enj commented Nov 10, 2017

/retest

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit d076bb5 into openshift:master Nov 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants