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

dynamic resource allocation: add DRA-managed resources to the PodResources API #113783

Closed
klueska opened this issue Nov 9, 2022 · 7 comments
Closed
Labels
area/kubelet kind/feature Categorizes issue or PR as related to a new feature. sig/node Categorizes an issue or PR as relevant to SIG Node. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@klueska
Copy link
Contributor

klueska commented Nov 9, 2022

What would you like to be added?

The PodResources API is used to query the full set of resources allocated to a pod and make them available to metrics servers, CNI plugins, etc.

With the introduction of DRA, we should also include the resources it manages as part of this API.

Why is this needed?

To allow metrics servers, CNI plugins, and the like to discover the set of DRA managed resources allocated to a pod.

@klueska klueska added the kind/feature Categorizes issue or PR as related to a new feature. label Nov 9, 2022
@k8s-ci-robot k8s-ci-robot added needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 9, 2022
@klueska
Copy link
Contributor Author

klueska commented Nov 9, 2022

/sig node

@k8s-ci-robot k8s-ci-robot added sig/node Categorizes an issue or PR as relevant to SIG Node. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Nov 9, 2022
@klueska klueska changed the title dynamic resource allocation: add resources managed by DRA to the PodResources API dynamic resource allocation: add DRA-managed to the PodResources API Nov 9, 2022
@ffromani
Copy link
Contributor

ffromani commented Nov 9, 2022

/triage accepted
I agree adding support for DRA makes sense for podresources API

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 9, 2022
@klueska klueska changed the title dynamic resource allocation: add DRA-managed to the PodResources API dynamic resource allocation: add DRA-managed resources to the PodResources API Nov 10, 2022
@bart0sh
Copy link
Contributor

bart0sh commented Nov 16, 2022

/area kubelet

@swatisehgal
Copy link
Contributor

/cc

@klueska
Copy link
Contributor Author

klueska commented Jan 9, 2023

/assign @moshe010

@k8s-ci-robot
Copy link
Contributor

@klueska: GitHub didn't allow me to assign the following users: moshe010.

Note that only kubernetes members with read permissions, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign @moshe010

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@klueska
Copy link
Contributor Author

klueska commented Jul 10, 2023

Completed by:
kubernetes/enhancements#3695

@klueska klueska closed this as completed Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubelet kind/feature Categorizes issue or PR as related to a new feature. sig/node Categorizes an issue or PR as relevant to SIG Node. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
Development

No branches or pull requests

5 participants