-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Change api version in examples #5030
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
Conversation
|
[test][extended:core] |
|
@jhadvig need to make sure none of the tests are reading that files (should not break anything, just want to make sure ;-) |
|
continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/5929/) |
|
@mfojtik yeah Im already doing checks |
|
I'd like a hold on this until after we've disabled #4992. Even after we disable that REST endpoint, we should keep at least a few v1beta3 objects as fixtures to make sure the client properly converts and sends to the proper API version. |
I agree; I'm actually stuck on #4992 at the moment tracking down an issue which was only exposed because of the v1beta3 example files used in |
|
@ironcladlou agreed. |
|
@jhadvig Bump |
|
@deads2k @ironcladlou I've updated the PR to only change the sample-app templates and we can change the rest when #4992 landes. |
|
Evaluated for origin test up to 693d92a |
|
LGTM [merge] |
|
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/3694/) (Image: devenv-rhel7_2501) |
|
Evaluated for origin merge up to 693d92a |
Merged by openshift-bot
@mfojtik @Kargakis PTAL