-
Notifications
You must be signed in to change notification settings - Fork 38.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
Automated cherry pick of #75952: tests: Solve backoff tests flakiness Cherry pick of #75952 on release-1.14. #75952: tests: Solve backoff tests flakiness #78858
Automated cherry pick of #75952: tests: Solve backoff tests flakiness Cherry pick of #75952 on release-1.14. #75952: tests: Solve backoff tests flakiness #78858
Conversation
The container status is not constant, and can change over time in the following order: - Running: When kubelet reports the Pod as running. This state is missable if the container finishes its command faster than kubelet getting to report this state. - Terminated: After the Container finished its command, it will enter the Terminated state, in which will remain for a short period of time, before kubelet will try to restart it. - Waiting: When kubelet has to wait for the backoff period to expire before actually restarting the container. Treating and handling each of these states when calculating the backoff period between container restarts will make the tests more reliable.
@BCLAU: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. 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. |
@BCLAU: This PR is not for the master branch but does not have the To approve the cherry-pick, please assign the patch release manager for the release branch by writing The list of patch release managers for each release can be found here. 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. |
/test pull-kubernetes-integration |
/assign @foxish |
Is there a reason the cherry pick automation was not used What is the master branch PR / commit from which this is picked? /hold |
Looks like this is coming from: |
I followed this guide: https://stupefied-goodall-e282f7.netlify.com/contributors/devel/cherry-picks/ I didn't know of any other guide. |
@kubernetes/sig-node-pr-reviews looking for lgtm/approve here or close |
@kubernetes/sig-testing-pr-reviews also looking for your input |
/lgtm |
/assign @foxish |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bclau, derekwaynecarr, foxish 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 |
Reason for cherry-picking:
The container status is not constant, and it fluctuates between
Running
,Terminated
, andWaiting
, and it is possible for the test to miss any of those transitions, resulting in the test being flaky. This PR eliminates it by treating all the states, rather than just theTerminated
state.Will help improve stability for: https://testgrid.k8s.io/sig-windows#aks-engine-azure-1-14-windows