Skip to content
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

UPSTREAM: 42491: make the system:authenticated group adder smarter #13247

Merged
merged 3 commits into from
Mar 8, 2017

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented Mar 6, 2017

The larger change didn't make 1.6, but the smaller change did. Revert the big one, use the little one.

@enj ptal

@deads2k
Copy link
Contributor Author

deads2k commented Mar 6, 2017

[test]

@deads2k
Copy link
Contributor Author

deads2k commented Mar 6, 2017

re[test]

@deads2k
Copy link
Contributor Author

deads2k commented Mar 7, 2017

[test]

@deads2k deads2k force-pushed the auth-03-proxy-groups branch from 3bb73c7 to 0818506 Compare March 7, 2017 13:28
@deads2k deads2k force-pushed the auth-03-proxy-groups branch from 0818506 to 6e20c55 Compare March 7, 2017 14:45
@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 6e20c55

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/905/) (Base Commit: 192e57e)

@enj
Copy link
Contributor

enj commented Mar 7, 2017

LGTM.

@deads2k
Copy link
Contributor Author

deads2k commented Mar 7, 2017

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 6e20c55

@openshift-bot
Copy link
Contributor

openshift-bot commented Mar 7, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/929/) (Base Commit: 93efd9b) (Image: devenv-rhel7_6052)

@openshift-bot openshift-bot merged commit 21527d7 into openshift:master Mar 8, 2017
@deads2k deads2k deleted the auth-03-proxy-groups branch August 3, 2017 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants