-
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
Fix clusterresourcequota annotations validation #10929
Fix clusterresourcequota annotations validation #10929
Conversation
os::cmd::try_until_text 'oc describe appliedclusterresourcequota/for-deads-by-annotation -n bar --as deads' "secrets.*1[0-9]" | ||
|
||
os::cmd::expect_success 'oc delete project foo' | ||
os::cmd::expect_success 'oc delete project bar' | ||
os::cmd::expect_success 'oc delete project asmail' |
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.
We should be doing cleanup in a error-tolerant subshell at the start of the suite so that this suite is re-entrant if a run fails before cleanup occurs. See other test/cmd
suites for an example.
[test] |
[test] flake on Service
|
4b2a330
to
12e331b
Compare
Cherry-picked again after @stevekuznetsov comments in the master PR. |
12e331b
to
2d4e88f
Compare
Evaluated for origin test up to 2d4e88f |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9166/) |
@liggitt good to merge here? |
yes, [merge] |
continuous-integration/openshift-jenkins/merge Evaluating for testing |
Merge does not work on this branch. |
Porting #10912 to
release-1.3
.