-
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] Deployment deployment should delete old replica sets {Kubernetes e2e suite} #28339
Comments
Failed: [k8s.io] Deployment deployment should delete old replica sets {Kubernetes e2e suite}
|
Hi, I wanted to report issue that after "deployment update" (I change image name with "kubectl set image deployment/..... ", and it does rolling update with creating a new replica-set) old replica-set that now has 0 pods, still exist in dashboard. But I guess this ticket addresses it already, right? |
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/kubernetes-e2e-gke/13392/ Failed: [k8s.io] Deployment deployment should delete old replica sets {Kubernetes e2e suite}
|
[FLAKE-PING] @janetkuo This flaky-test issue would love to have more attention... |
1 similar comment
[FLAKE-PING] @janetkuo This flaky-test issue would love to have more attention... |
[FLAKE-PING] @janetkuo This flaky-test issue would love to have more attention. |
[FLAKE-PING] @janetkuo This flaky-test issue would love to have more attention. |
@davidopp so this fix will be released in 1.4? 👍 |
This is not an issue and unrelated to the current test flake issue. If you want to prune older replica sets, use revisionHistoryLimit=0 but note that you won't be able to rollback to an older revision if you ever need to. |
Thanks @Kargakis , i didn't know it's expected behavior. Now it makes sense :) |
Should be fixed by #33030 |
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/kubernetes-e2e-gke/10295/
Failed: [k8s.io] Deployment deployment should delete old replica sets {Kubernetes e2e suite}
The text was updated successfully, but these errors were encountered: