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

flex karpenter node-pool configurations #775

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

myaser
Copy link
Member

@myaser myaser commented Jun 3, 2024

  • make the instanceType column non mandatory for karpenter pools
  • parse a node pool config item "requirements" to allow injecting granular karpenter pool requirements
    example:
  - config_items:
      taints: nvidia.com/gpu=present:NoSchedule
      requirements: |
        - key: karpenter.k8s.aws/instance-gpu-manufacturer
          operator: In
          values: 
            - nvidia
    discount_strategy: null
    instance_types: null
    max_size: null
    min_size: null
    name: karpenter-gpu
    profile: worker-karpenter

related to zalando-incubator/kubernetes-on-aws#7617

@myaser myaser force-pushed the flex-karpenter-node-pool-configs branch 2 times, most recently from 59102c6 to 1e01ccf Compare June 3, 2024 10:08
Signed-off-by: Mahmoud Gaballah <mahmoud.gaballah@zalando.de>
@myaser myaser force-pushed the flex-karpenter-node-pool-configs branch from 1e01ccf to e58e3cb Compare June 3, 2024 12:23
@myaser myaser requested a review from linki June 20, 2024 12:26
@demonCoder95
Copy link
Member

👍

1 similar comment
@myaser
Copy link
Member Author

myaser commented Jul 4, 2024

👍🏻

@myaser myaser merged commit 4615c1a into master Jul 4, 2024
7 of 9 checks passed
@myaser myaser deleted the flex-karpenter-node-pool-configs branch July 4, 2024 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants