feat: add support for instance generation, accelerated devices, instance class, and network out#44
Merged
AshleyDumaine merged 12 commits intomainfrom Jan 26, 2026
Merged
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #44 +/- ##
==========================================
+ Coverage 66.85% 67.08% +0.23%
==========================================
Files 37 37
Lines 2311 2315 +4
==========================================
+ Hits 1545 1553 +8
+ Misses 616 613 -3
+ Partials 150 149 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
f05f53a to
2033a9f
Compare
94eb128 to
91c8dbc
Compare
91c8dbc to
d95be11
Compare
Contributor
Author
|
It would seem we can't filter by region for ListTypes? We apparently would just have to grab everything and filter client side. EDIT: There's no reliable way to do this so we instead just make sure the error is clear to the user in the NodeClaim status. |
… out field supported by LinodeType data, fix instance class
rosskirkpat
approved these changes
Jan 23, 2026
Contributor
|
Could we add some usage docs on the new fields? |
komer3
reviewed
Jan 23, 2026
fix the CRD versioning to indicate this is still alpha
feat: add support for instance disk and gpu name requirements
komer3
approved these changes
Jan 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
With this it should be possible to specify the Linode Class (nanode, standard, highmem, dedicated, gpu), the number of accelerated devices, and the instance generation. (g6, g7, g8).
Testing
Follow https://github.com/linode/karpenter-provider-linode?tab=readme-ov-file#using-karpenter but for the NodePool add an extra requirement like the generation (this asks for the shiny new g8 instance types only):
When scaling the inflate deployment in the README example, see if you picked a region that has the plan type supported.
us-southeastdidn't and should reflect that error in the nodeclaim when you check its status:If you want to check the other 2 new supported fields, scale inflate to 0 (make sure the nodeclaim goes away), remove the generation requirement from the NodePool, and sub in:
then scale back up to request dedicated instances only. Same thing for accelerated devices (again, this might not work in the region you picked, just check the nodeclaim):
or network out:
From what I can tell, you can't just edit the NodePool and have the NodeClaim automatically update its list of instances to choose from.