-
Notifications
You must be signed in to change notification settings - Fork 232
Use apps
API group for deployments
#1813
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
Use apps
API group for deployments
#1813
Conversation
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 for 3.7
[test] |
1d37f6c
to
325bf13
Compare
Rebased |
325bf13
to
d10daf0
Compare
Rebased |
d10daf0
to
a60cbe7
Compare
Evaluated for origin web console test up to a60cbe7 |
Origin Web Console Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_web_console/8/) (Base Commit: e3e7ba7) (PR Branch Commit: a60cbe7) |
[merge] |
Evaluated for origin web console merge up to a60cbe7 |
Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/22/) (Base Commit: 0d92433) (PR Branch Commit: a60cbe7) |
See #255
@jwforres for 3.7 :)
@Kargakis