Skip to content
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

Karpenter should warn upon using likely incorrect labels in NodePools #5521

Open
tzneal opened this issue Jan 24, 2024 · 1 comment
Open
Labels
feature New feature or request good-first-issue Good for newcomers

Comments

@tzneal
Copy link
Contributor

tzneal commented Jan 24, 2024

Description

What problem are you trying to solve?

Adding a requirement like this to your NodePool will cause nodes to be launched with a random label from this set:

 - key: "eks.amazonaws.com/capacityType"
    operator: In
    values: [ "ON_DEMAND", "SPOT" ]

This should at least produce a warning in the controller as the current behavior can be confusing.

How important is this feature to you?

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment
@tzneal tzneal added feature New feature or request needs-triage Issues that need to be triaged labels Jan 24, 2024
@jonathan-innis jonathan-innis removed the needs-triage Issues that need to be triaged label Jan 24, 2024
@tzneal tzneal added the good-first-issue Good for newcomers label Jan 29, 2024
@muckelba
Copy link

We've stumbled across the same issue. The eks.amazonaws.com/capacityType label is randomly wrong when having it in the karpenter requirements. If it's not in it, its not set at all, so i think using karpenter.sh/capacity-type instead is the best workaround for now.

Is there anything karpenter can do to prevent this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request good-first-issue Good for newcomers
Projects
None yet
3 participants