Skip to content

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented Sep 6, 2017

It's going to take a couple runs here to find an fix the openshift roles that need updating.

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 6, 2017
@openshift-merge-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k

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

@openshift-merge-robot openshift-merge-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 6, 2017
@stevekuznetsov
Copy link
Contributor

/unassign

@deads2k
Copy link
Contributor Author

deads2k commented Sep 6, 2017

/retest

@deads2k
Copy link
Contributor Author

deads2k commented Sep 7, 2017

Jordan lgtm'd upstream. This fixes permissions on our controllers. Tagging

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

Automatic merge from submit-queue (batch tested with PRs 15994, 16182, 16190, 16174)

@openshift-merge-robot openshift-merge-robot merged commit 52567ea into openshift:master Sep 7, 2017
@deads2k deads2k deleted the server-43-deletion branch January 24, 2018 14:33
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. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants