-
Notifications
You must be signed in to change notification settings - Fork 958
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
docs: update karpenter docs around soft affinity behavior #6172
docs: update karpenter docs around soft affinity behavior #6172
Conversation
✅ Deploy Preview for karpenter-docs-prod ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thanks for the docs contribution @cnmcavoy! Got a few thoughts. |
Pull Request Test Coverage Report for Build 9412990237Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
cc3879c
to
736bb46
Compare
736bb46
to
6b556ce
Compare
6b556ce
to
924084e
Compare
This PR has been inactive for 14 days. StaleBot will close this stale PR after 14 more days of inactivity. |
924084e
to
95d3186
Compare
95d3186
to
b0d7063
Compare
59a04c6
to
d05f567
Compare
d05f567
to
ed32d0b
Compare
I hope I managed to address everything, many good suggestions. |
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 🚀
Fixes #N/A
Description
Documents Karpenter behavior around soft affinities.
How was this change tested?
Docs only change.
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.