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

feat: support ContainerOptimizedOS resolving #14

Merged
merged 1 commit into from
Mar 2, 2025

Conversation

jwcesign
Copy link
Contributor

@jwcesign jwcesign commented Mar 2, 2025

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


@jwcesign
Copy link
Contributor Author

jwcesign commented Mar 2, 2025

Testing result:

apiVersion: karpenter.k8s.gcp/v1alpha1
kind: GCENodeClass
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"karpenter.k8s.gcp/v1alpha1","kind":"GCENodeClass","metadata":{"annotations":{},"name":"defaultnodeclass"},"spec":{"imageSelectorTerms":[{"alias":"ContainerOptimizedOS"}]}}
  creationTimestamp: "2025-03-02T09:07:00Z"
  finalizers:
  - karpenter.k8s.gcp/termination
  generation: 1
  name: defaultnodeclass
  resourceVersion: "71985008"
  uid: 300f7476-c38c-4f40-b6ac-61c16c6fd6f4
spec:
  imageSelectorTerms:
  - alias: ContainerOptimizedOS
status:
  conditions:
  - lastTransitionTime: "2025-03-02T09:07:00Z"
    message: ""
    observedGeneration: 1
    reason: ImagesReady
    status: "True"
    type: ImagesReady
  - lastTransitionTime: "2025-03-02T09:07:00Z"
    message: ""
    observedGeneration: 1
    reason: Ready
    status: "True"
    type: Ready
  images:
  - requirements:
    - key: kubernetes.io/arch
      operator: In
      values:
      - amd64
    - key: karpenter.k8s.gcp/instance-gpu-count
      operator: Exists
    sourceImage: projects/gke-node-images/global/images/gke-1309-gke1046000-cos-113-18244-291-9-c-nvda
  - requirements:
    - key: kubernetes.io/arch
      operator: In
      values:
      - amd64
    - key: karpenter.k8s.gcp/instance-gpu-count
      operator: DoesNotExist
    sourceImage: projects/gke-node-images/global/images/gke-1309-gke1046000-cos-113-18244-291-9-c-pre
  - requirements:
    - key: kubernetes.io/arch
      operator: In
      values:
      - arm64
    - key: karpenter.k8s.gcp/instance-gpu-count
      operator: DoesNotExist
    sourceImage: projects/gke-node-images/global/images/gke-1309-gke1046000-cos-arm64-113-18244-291-9-c-pre

Signed-off-by: jwcesign <jwcesign@cloudpilot.ai>
@jwcesign jwcesign merged commit 8f1df04 into cloudpilot-ai:main Mar 2, 2025
1 check passed
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.

1 participant