Skip to content

feat: determine cluster ID from cluster name (label) if in LKE mode#42

Merged
AshleyDumaine merged 3 commits intomainfrom
cluster-id-from-name
Jan 22, 2026
Merged

feat: determine cluster ID from cluster name (label) if in LKE mode#42
AshleyDumaine merged 3 commits intomainfrom
cluster-id-from-name

Conversation

@AshleyDumaine
Copy link
Contributor

@AshleyDumaine AshleyDumaine commented Jan 22, 2026

At startup when creating the operator, if in LKE mode (default) we automatically determine the cluster ID from the name and fail if there is not one match returned.
Other providers take a cluster name rather than an ID if using managed K8s, so this is to follow the standard convention.

Also adding a GHA change to make sure we build and push images for main so the latest tag exists for linode/karpenter-provider-linode so the README steps work without needing to build and push the image first + update the deployment image.

@codecov-commenter
Copy link

codecov-commenter commented Jan 22, 2026

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.99%. Comparing base (3f03a8a) to head (c7e7cf0).

Files with missing lines Patch % Lines
pkg/operator/operator.go 0.00% 14 Missing ⚠️
pkg/fake/linodeapi.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   67.37%   66.99%   -0.38%     
==========================================
  Files          37       37              
  Lines        2296     2306      +10     
==========================================
- Hits         1547     1545       -2     
- Misses        599      612      +13     
+ Partials      150      149       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AshleyDumaine AshleyDumaine merged commit a70759b into main Jan 22, 2026
5 checks passed
@AshleyDumaine AshleyDumaine deleted the cluster-id-from-name branch January 22, 2026 21:29
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.

3 participants