Skip to content

migration: add setting to enable nvidia device plugin#4856

Open
maherthomsi wants to merge 1 commit into
bottlerocket-os:developfrom
maherthomsi:nvidia-migrations
Open

migration: add setting to enable nvidia device plugin#4856
maherthomsi wants to merge 1 commit into
bottlerocket-os:developfrom
maherthomsi:nvidia-migrations

Conversation

@maherthomsi

Copy link
Copy Markdown
Contributor

Description of changes:
Adds the nvidia-k8s-device-plugin-enabled settings migration.

This migration supports the new settings.kubelet-device-plugins.nvidia.enabled field added in bottlerocket-settings-sdk#135 and bottlerocket-core-kit#914.

  • Forward (upgrade): No-op. The enabled field defaults to None (treated as enabled), so existing data is compatible.
  • Backward (downgrade): Removes settings.kubelet-device-plugins.nvidia.enabled from the datastore so older versions don't encounter an unknown field.

Testing done:

  • cargo test passes for all workspace crates including nvidia-k8s-device-plugin-enabled
  • Full TUF upgrade/downgrade test on g4dn.xlarge (aws-k8s-1.35-nvidia, x86_64):
    • Built v1.62.0 base AMI from release tag
    • Built v1.63.0 image with migration, published to custom TUF repo (S3 + CloudFront)
    • Launched v1.62.0 instance, upgraded to v1.63.0 via updog update -i 1.63.0 -r -n - settings preserved (forward no-op confirmed)
    • Set settings.kubelet-device-plugins.nvidia.enabled=true on v1.63.0
    • Rolled back to v1.62.0 via signpost rollback-to-inactive + reboot - enabled key successfully removed by backward migration, all other settings intact

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

@maherthomsi maherthomsi marked this pull request as ready for review June 8, 2026 21:19
@maherthomsi maherthomsi marked this pull request as draft June 8, 2026 21:20
@maherthomsi maherthomsi force-pushed the nvidia-migrations branch 7 times, most recently from 1d1e10c to 0d98e33 Compare June 9, 2026 22:47
@maherthomsi maherthomsi marked this pull request as ready for review June 9, 2026 23:18
Signed-off-by: Maher Homsi <maherhom@amazon.com>
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.

4 participants