-
Notifications
You must be signed in to change notification settings - Fork 1.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
ci-operator/jobs/openshift/release: Drop --rehearsal-limit #12966
ci-operator/jobs/openshift/release: Drop --rehearsal-limit #12966
Conversation
Folks have been tuning this since b0d60e3 (Bump the limit of rehearsed jobs, 2019-08-19, openshift#4789), most recently in 3f0040e (Raise rehearsal-limit to 45 temporarily, 2020-02-03, openshift#6999). But with [1], we no longer need to fuss with this setting in order to see rehearsals for changes that touch lots of jobs, so let it fall back to pj-rehearse's default of 15. [1]: openshift/ci-tools#1315
|
Seems like a temporary issue which has since cleared up. /retest |
All green :) /assign @stevekuznetsov |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: stevekuznetsov, wking The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
We're waiting on openshift/ci-tools#1317 |
landed and published. /hold cancel |
@wking: Updated the following 2 configmaps:
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Folks have been tuning this since b0d60e3 (#4789), most recently in 3f0040e (#6999). But with openshift/ci-tools#1315, we no longer need to fuss with this setting in order to see rehearsals for changes that touch lots of jobs, so let it fall back to
pj-rehearse
's default of 15.