-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
remove featuregate DevicePlugins #41095
Conversation
Thanks @carlory for the PR. |
LGTM label has been added. Git tree hash: 497c7b0667a6b2b03422af18067d717bf4083568
|
In https://k8s.io/docs/concepts/extend-kubernetes/compute-storage-net/device-plugins/, it is still said that
Do we need to revise this? |
1 similar comment
In https://k8s.io/docs/concepts/extend-kubernetes/compute-storage-net/device-plugins/, it is still said that
Do we need to revise this? |
@Shubham82 The graduation of the API and the DeviceManager component is split, according to this discussion https://kubernetes.slack.com/archives/C0BP8PW9G/p1667481109075849. The graduation of the API is tracking, see kubernetes/kubernetes#113587. @tengqm @Shubham82 what do you think ?
|
@carlory can we mention the issue k8s/k8s #113587 here, for better understanding of readers i.e. we Device Plugin API didn't graduate alongside the DeviceManager. |
@Shubham82 done. |
content/en/docs/concepts/extend-kubernetes/compute-storage-net/device-plugins.md
Outdated
Show resolved
Hide resolved
/sig node |
/cc @tengqm @Shubham82 |
Thanks. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tengqm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@carlory, Rebase your changes. |
@Shubham82 done |
JFI: This PR also removed the graduated feature gate |
Thanks, @carlory |
LGTM label has been added. Git tree hash: 19b8fa7392949c7a7a860e0433442e26557733d3
|
due to kubernetes/kubernetes#117656