-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
move unrelated extended tests out of [images] #10481
Conversation
@PI-Victor it looks ok to me. @deads2k ? |
Still a little annoyed that we got kicked out of using our own API group for a tag, but not annoyed enough to fight for it. [merge] |
Evaluated for origin merge up to 5797fdf |
continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8036/) |
@deads2k was just trying to figure out if there's an issue open, thank you! |
* create a different folder imageapis * move limitrange_admission and quote_admission tests out * change tags to imageapis
Evaluated for origin test up to 4f26aa7 |
[testextended][extended:core(imageapis)] |
Evaluated for origin testextended up to 4f26aa7 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8157/) |
Origin Action Required: Please contact #openshift-dev to have this pull request manually reviewed and tested |
continuous-integration/openshift-jenkins/testextended FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin_extended/432/) (Extended Tests: core(imageapis)) |
this pr needed a slight tweak, redoing here: |
fixes: #10375
@mfojtik is this ok, am i missing something?
cc @bparees