-
Notifications
You must be signed in to change notification settings - Fork 960
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
ci: don't remove cluster with unhealthy mng #5746
Conversation
✅ Deploy Preview for karpenter-docs-prod canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/karpenter snapshot
Pull Request Test Coverage Report for Build 8118291183Details
💛 - Coveralls |
Snapshot successfully published to
|
fb352b4
to
8d84f13
Compare
8d84f13
to
97d5ec3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
This reverts commit 4b1d4e6.
Fixes #N/A
Description
Skips the cluster cleanup process if the kubelet is not ready on one of the MNG nodes. This is to help diagnose the following failure: https://github.com/aws/karpenter-provider-aws/actions/runs/8032556917/job/21942085645.
How was this change tested?
/karpenter snapshot
Does this change impact docs?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.