Update nodedeletiontimeout for tinkerbell to 30s#10283
Update nodedeletiontimeout for tinkerbell to 30s#10283panktishah26 wants to merge 1 commit intoaws:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
2b4c04c to
5ef1768
Compare
5ef1768 to
3ece945
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #10283 +/- ##
==========================================
- Coverage 70.01% 70.01% -0.01%
==========================================
Files 680 680
Lines 41748 41751 +3
==========================================
+ Hits 29230 29231 +1
- Misses 10721 10723 +2
Partials 1797 1797 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/test eks-anywhere-presubmit |
|
/test eks-anywhere-e2e-presubmit |
|
@panktishah26: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
Description of changes:
For tinkerbell, we are seeing e2e tests failures with the error message like below,
this happens when capt takes more than 10s (default node deletion timeout) to delete the machine corresponding to node.
Increasing
nodedeletiontimeoutto 30s for tinkerbell provider to overcome this issue.Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.