-
Notifications
You must be signed in to change notification settings - Fork 40k
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
[k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite} #31066
Comments
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/kubernetes-e2e-gke/13804/ Failed: [k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite}
|
Failed: [k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite}
Happened on a presubmit run in #30652. |
Looks like the kubectl doesn't handle the node update conflict. @kevin-wangzefeng do you mind if I take the issue since you are not working on it? |
Failed: [k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite}
Happened on a presubmit run in #31156. |
@caesarxuchao sorry for the replying late, I just sent #31578 to fix this. |
Thanks, @kevin-wangzefeng! But there are two variations of the flake here. Will the retry fix this variation: #31066 (comment)? Do you have a theory why the taint is missing? |
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/kubernetes-e2e-gce/22386/ Failed: [k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite}
|
[FLAKE-PING] @caesarxuchao @kevin-wangzefeng This flaky-test issue would love to have more attention... |
Thanks, @kevin-wangzefeng! But there are two variations of the flake here. Will the retry fix this variation: #31066 (comment)? Do you have a theory why the taint is missing? |
@caesarxuchao Thanks for reminding that, I guess there is a mistake in the describe #L2577-L2590. I'm taking a look. |
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/kubernetes-e2e-gke/14450/ Failed: [k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite}
|
[FLAKE-PING] @caesarxuchao @kevin-wangzefeng This flaky-test issue would love to have more attention. |
Failed: [k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite}
Happened on a presubmit run in #31600. |
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/kubernetes-e2e-gce/21985/
Failed: [k8s.io] Kubectl client [k8s.io] Kubectl taint should remove all the taints with the same key off a node {Kubernetes e2e suite}
The text was updated successfully, but these errors were encountered: