aws / karpenter-provider-aws

Karpenter is a Kubernetes Node Autoscaler built for flexibility, performance, and simplicity.
https://karpenter.sh
Apache License 2.0
6.15k stars 849 forks source link

chore: Use client-go `scheme.Scheme` #6419

Closed jonathan-innis closed 5 days ago

jonathan-innis commented 6 days ago

Fixes #N/A

Description

How was this change tested?

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.

netlify[bot] commented 6 days ago

Deploy Preview for karpenter-docs-prod canceled.

Name Link
Latest commit 54f47f6c3dbf34900106bd0f6b24225d54e39e77
Latest deploy log https://app.netlify.com/sites/karpenter-docs-prod/deploys/667d0c1bbe836c00073ceef9
github-actions[bot] commented 6 days ago

Snapshot successfully published to oci://021119463062.dkr.ecr.us-east-1.amazonaws.com/karpenter/snapshot/karpenter:0-27f479f1ddb56ac57b1dbce17da80a11ef899e19. To install you must login to the ECR repo with an AWS account:

aws ecr get-login-password --region us-east-1 | docker login --username AWS --password-stdin 021119463062.dkr.ecr.us-east-1.amazonaws.com

helm upgrade --install karpenter oci://021119463062.dkr.ecr.us-east-1.amazonaws.com/karpenter/snapshot/karpenter --version "0-27f479f1ddb56ac57b1dbce17da80a11ef899e19" --namespace "kube-system" --create-namespace \
  --set "settings.clusterName=${CLUSTER_NAME}" \
  --set "settings.interruptionQueue=${CLUSTER_NAME}" \
  --set controller.resources.requests.cpu=1 \
  --set controller.resources.requests.memory=1Gi \
  --set controller.resources.limits.cpu=1 \
  --set controller.resources.limits.memory=1Gi \
  --wait
coveralls commented 5 days ago

Pull Request Test Coverage Report for Build 9692019868

Details


Files with Coverage Reduction New Missed Lines %
pkg/providers/amifamily/ami.go 1 90.56%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 9691760501: -0.002%
Covered Lines: 5790
Relevant Lines: 7319

💛 - Coveralls
jonathan-innis commented 5 days ago

Closing in favor of a different change in kubernetes-sigs/karpenter