Skip to content

Comments

Changes to support kata device plugin and mode#2142

Merged
cdesiniotis merged 2 commits intoNVIDIA:mainfrom
jojimt:kata
Feb 24, 2026
Merged

Changes to support kata device plugin and mode#2142
cdesiniotis merged 2 commits intoNVIDIA:mainfrom
jojimt:kata

Conversation

@jojimt
Copy link
Contributor

@jojimt jojimt commented Feb 20, 2026

Description

Kata/Coco uses a new sandbox device plugin. We need to update the operator to natively support it. To this end,
the following has been added to the cluster policy:

  • a mode field to sandboxWorkloads, with ["kube-virt", "kata"] as valid values
  • a kataDevicePlugin section

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

@copy-pr-bot
Copy link

copy-pr-bot bot commented Feb 20, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@tariq1890
Copy link
Contributor

/ok to test ae95274

@jojimt jojimt force-pushed the kata branch 2 times, most recently from 15bb6be to 6b1ceaa Compare February 20, 2026 21:08
@jojimt jojimt marked this pull request as ready for review February 20, 2026 21:09
@tariq1890
Copy link
Contributor

/ok to test 6b1ceaa

@jojimt jojimt force-pushed the kata branch 2 times, most recently from 7a6c373 to 1e8bf18 Compare February 20, 2026 22:59
@tariq1890
Copy link
Contributor

/ok to test 1e8bf18

@tariq1890
Copy link
Contributor

Let's update the sample clusterpolicy JSON here to reflect the latest clusterpolicy API updates

@jojimt jojimt force-pushed the kata branch 2 times, most recently from da03feb to aa9f18b Compare February 23, 2026 21:08
@jojimt jojimt force-pushed the kata branch 2 times, most recently from 21480b8 to fedaf1a Compare February 23, 2026 22:30
@tariq1890
Copy link
Contributor

/ok to test fedaf1a

1 similar comment
@cdesiniotis
Copy link
Contributor

/ok to test fedaf1a

@@ -38,6 +38,7 @@ metadata:
"sandboxWorkloads": {
"enabled": false,
"defaultWorkload": "container"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing a comma at the end of L40

Signed-off-by: Joji Mekkattuparamban <jojim@nvidia.com>
@cdesiniotis
Copy link
Contributor

/ok to test 738d242

Signed-off-by: Joji Mekkattuparamban <jojim@nvidia.com>
@cdesiniotis
Copy link
Contributor

/ok to test d6bb71f

@cdesiniotis cdesiniotis merged commit 2d3ab99 into NVIDIA:main Feb 24, 2026
8 of 15 checks 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.

4 participants