Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

prepare device plugin API changes for CDI injection #1457

Open
mythi opened this issue Jun 19, 2023 · 4 comments
Open

prepare device plugin API changes for CDI injection #1457

mythi opened this issue Jun 19, 2023 · 4 comments
Assignees

Comments

@mythi
Copy link
Contributor

mythi commented Jun 19, 2023

Kubernetes KEP4009: Add CDI Devices to device plugin API got approved.

It will be added as an alpha feature but we could look into adding the support to our framework and see what use-cases it enables (e.g., FPGA deallocate)

@bart0sh
Copy link
Member

bart0sh commented Jan 11, 2024

Considering that CDI support for Device Plugins has been graduated to Beta in Kubernetes 1.29, this task becomes more important. Moreover, it can be a show stopper for graduating CDI support for DP to GA as this requires Gathering feedback from at least 2 device plugin vendors that CDI support works for them.

@mythi @tkatila Can we prioritise this issue?

@tkatila
Copy link
Contributor

tkatila commented Jan 11, 2024

@bart0sh patches are welcome. :)

Is there a time frame for the feedback?

@bart0sh
Copy link
Member

bart0sh commented Feb 15, 2024

I'm going to add CDI support for FPGA plugin in near future.

@bart0sh bart0sh self-assigned this Mar 22, 2024
@bart0sh
Copy link
Member

bart0sh commented May 22, 2024

@tkatila

Is there a time frame for the feedback?

I'm sorry for missing your question. We're hoping to get CDI support for device plugins graduated to GA in the next Kubernetes release.
It would help a lot to achieve this if this PR is merged.

When it's merged, I'll update the feedback info of the KEP and you can add your feedback there as review comments. But first we should merge the implementation.

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

No branches or pull requests

3 participants