-
Notifications
You must be signed in to change notification settings - Fork 934
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
chore: Update GetSupportedNodeClasses to return []status.Object #6414
Conversation
✅ Deploy Preview for karpenter-docs-prod canceled.
|
3094e46
to
8881d2d
Compare
Pull Request Test Coverage Report for Build 9719173984Details
💛 - Coveralls |
8881d2d
to
4cc2bd2
Compare
Pull Request Test Coverage Report for Build 9753011005Details
💛 - Coveralls |
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 🚀
4cc2bd2
to
55bad29
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.
/karpenter snapshot
Snapshot successfully published to
|
55bad29
to
044d7bf
Compare
Pull Request Test Coverage Report for Build 9813097959Details
💛 - Coveralls |
044d7bf
to
04165e8
Compare
Pull Request Test Coverage Report for Build 9813218711Details
💛 - Coveralls |
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
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
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
Snapshot successfully published to
|
04165e8
to
aaca676
Compare
Pull Request Test Coverage Report for Build 9844631660Details
💛 - Coveralls |
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
Snapshot successfully published to
|
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
This PR adds a change to return
[]status.object
fromGetSupportedNodeClasses
How was this change tested?
Tested on local cluster
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.