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

The efficiency of synchronizing Pod status too slow. #123875

Open
FengyunPan2 opened this issue Mar 12, 2024 · 5 comments
Open

The efficiency of synchronizing Pod status too slow. #123875

FengyunPan2 opened this issue Mar 12, 2024 · 5 comments
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/node Categorizes an issue or PR as relevant to SIG Node.

Comments

@FengyunPan2
Copy link
Contributor

FengyunPan2 commented Mar 12, 2024

What would you like to be added?

Use a single List api instead of multiple Get api.

Why is this needed?

My k8s node is a big machine(500C CPU & 2000GB Memory), the maxPods of kubelet could be set to 200+.
When I delete 200+ pods or create 200+ pods, the synchronization of Pod status by the kubelet becomes too slow(about 230s).
Making a large number of GET API calls can indeed increase the load on the API server.

@FengyunPan2 FengyunPan2 added the kind/feature Categorizes issue or PR as related to a new feature. label Mar 12, 2024
@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 Mar 12, 2024
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

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.

@FengyunPan2
Copy link
Contributor Author

/assign

@FengyunPan2 FengyunPan2 changed the title Improve the efficiency of synchronizing Pod status The efficiency of synchronizing Pod status too slow. Mar 12, 2024
FengyunPan2 added a commit to FengyunPan2/kubernetes that referenced this issue Mar 12, 2024
Use a single List api instead of multiple Get api.
About: kubernetes#123875
@neolit123
Copy link
Member

/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 Mar 12, 2024
FengyunPan2 added a commit to FengyunPan2/kubernetes that referenced this issue Mar 12, 2024
Use a single List api instead of multiple Get api.
About: kubernetes#123875
FengyunPan2 added a commit to FengyunPan2/kubernetes that referenced this issue Mar 12, 2024
Use a single List api instead of multiple Get api.
About: kubernetes#123875
@ffromani
Copy link
Contributor

do we have more precise numbers about the (in)efficiency? for example, a graph of pod count vs execution time

@matthyx
Copy link
Contributor

matthyx commented Mar 13, 2024

do we have more precise numbers about the (in)efficiency? for example, a graph of pod count vs execution time

I agree with @ffromani it would be nice to see how the new patch improves things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/node Categorizes an issue or PR as relevant to SIG Node.
Projects
None yet
Development

No branches or pull requests

5 participants