-
Notifications
You must be signed in to change notification settings - Fork 828
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
feat: Add evictionSoft
support to kubeletConfiguration
#2514
Conversation
✅ Deploy Preview for karpenter-docs-prod canceled.
|
ce79683
to
b08bc78
Compare
b08bc78
to
e813f28
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.
nice! just a few minor nits! Overall, very nice!
fa4c6fa
to
40d1a9d
Compare
Good comments! @bwagner5 |
40d1a9d
to
e778c82
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! nice 🔥
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.
nice catch! and bonus points, is more readable now :D
Fixes #
Description
evictionSoft
support intokubeletConfiguration
evictionSoft
,evictionSoftGracePeriod
andevictionMaxPodGracePeriod
How was this change tested?
make test
Does this change impact docs?
Release Note
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.