Skip to content

Add Kubernetes Client QPS and Burst Configuration#1013

Merged
benashz merged 14 commits intomainfrom
VAULT-33351/k8s-qps-burst-config
Mar 4, 2025
Merged

Add Kubernetes Client QPS and Burst Configuration#1013
benashz merged 14 commits intomainfrom
VAULT-33351/k8s-qps-burst-config

Conversation

@jaireddjawed
Copy link
Collaborator

@jaireddjawed jaireddjawed commented Feb 12, 2025

In clusters with many VaultStaticSecrets, the default Kubernetes client QPS of 5 and Burst of 10 is insufficient, leading it to throttle its communications with the Kubernetes API. This can cause brand-new deployments to roll out very slowly as their pods wait for their Kubernetes secrets to be created from their VaultStaticSecrets.

This PR addresses this by allowing users to change the Kubernetes client QPS and Burst values through environment variables/cli arguments and adds them as chart values.

@jaireddjawed jaireddjawed self-assigned this Feb 12, 2025
@jaireddjawed jaireddjawed force-pushed the VAULT-33351/k8s-qps-burst-config branch from 768eeb8 to 7e3f6fd Compare February 12, 2025 10:27
@jaireddjawed jaireddjawed marked this pull request as ready for review February 12, 2025 18:50
@jaireddjawed jaireddjawed requested a review from a team as a code owner February 12, 2025 18:50
@jaireddjawed jaireddjawed requested a review from benashz February 12, 2025 18:50
@jaireddjawed jaireddjawed requested a review from benashz February 20, 2025 17:45
@benashz benashz added this to the v0.10.0 milestone Mar 3, 2025
@benashz benashz merged commit 3253310 into main Mar 4, 2025
49 checks passed
@benashz benashz deleted the VAULT-33351/k8s-qps-burst-config branch March 4, 2025 15:24
@luislongom
Copy link

"In clusters with many VaultStaticSecrets..."

What does "many" mean?
https://discuss.hashicorp.com/t/clarify-changelog-entry-for-v0-10-0-add-kubernetes-client-qps-and-burst-configuration/76010

@sergeyshevch
Copy link

@luislongom For us we started to get issues at smth around 2k+ vault secrets in single cluster

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

Successfully merging this pull request may close these issues.

4 participants