Closed razo7 closed 4 months ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: razo7
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test 4.15-openshift-e2e
/test 4.15-openshift-e2e There are more (possible) changes for the e2e testing, but it can be done afterward and mostly when there are maintenance events.
/test 4.15-openshift-e2e
@razo7: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/4.15-openshift-e2e | 410b51c555dc5b1419b43d7d82db6074c51856f3 | link | true | /test 4.15-openshift-e2e |
Full PR test history. Your PR dashboard.
Closing this PR as its goal is minor, it introduces a possible longer reconciliation (while we strive for a short reconcile), an API change, and the gain doesn't worth the effort
What this PR does / why we need it:
Adds a new Spec field for the pod's eviction timeout. Configurable pod's eviction drain/deletion timeout for the user, while previously it was set to 30s (the default value).
Changes made
Which issue(s) this PR fixes:
ECOPROJECT-1668
Test plan
Add e2e test for different
evicitonTimeout
values - high (50s) vs. low (10s)