Skip to content

Conversation

marseel
Copy link
Contributor

@marseel marseel commented Nov 28, 2023

Default QPS for kubelet has been increased in 1.27 k8s to 50 QPS. https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.27.md under "Bump default API QPS limits for Kubelet.
Before kubelet had 5 QPS the same as agent.
Still when agent needs to create CEP and Identity for a pod, it has to make two API calls so potentially pod startup latency was higher than expected if node was experiancing high churn of pods with new identities.
Now with recent change of kubelet, there is even higher chance of that to happen - therefore we should increase QPS.

Default client-go QPS and burst in agent and operator have been increased to 10 and 20 respectively for k8s versions 1.27+

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 28, 2023
@marseel marseel force-pushed the increase_qps_for_client_go branch 2 times, most recently from 76b2ed0 to bc1079c Compare November 29, 2023 16:05
Default QPS for kubelet has been increased in 1.27 k8s to 50 QPS.
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.27.md
under "Bump default API QPS limits for Kubelet.
Before kubelet had 5 QPS the same as agent.
Still when agent needs to create CEP and Identity for a pod, it has to
make two API calls so potentially pod startup latency was higher than
expected if node was experiancing high churn of pods with new
identities.
Now with recent change of kubelet, there is even higher chance of that
to happen - therefore we should increase QPS.

Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
@marseel marseel force-pushed the increase_qps_for_client_go branch from bc1079c to f961bc6 Compare November 29, 2023 19:44
@marseel
Copy link
Contributor Author

marseel commented Nov 29, 2023

/test

@marseel marseel added the release-note/minor This PR changes functionality that users may find relevant to operating Cilium. label Nov 30, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 30, 2023
@marseel marseel marked this pull request as ready for review November 30, 2023 12:11
@marseel marseel requested review from a team as code owners November 30, 2023 12:11
@marseel marseel requested review from youngnick and gandro November 30, 2023 12:11
Copy link
Member

@gandro gandro left a comment

Choose a reason for hiding this comment

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

Helm looks good to me, thanks!

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 4, 2023
@youngnick youngnick added this pull request to the merge queue Dec 4, 2023
Merged via the queue into cilium:main with commit 5ef0f10 Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants