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

Add Kubernetes client QPS and Burst configuration support. #998

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hyperbolist
Copy link

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 change exposes the operator's Kubernetes client QPS and Burst configuration via cli arguments and environment variables, as well as explicitly via chart values.

  • adds --kube-client-qps and --kube-client-burst cli args
  • adds VSO_KUBE_CLIENT_QPS and VSO_KUBE_CLIENT_BURST env var support
  • adds .Values.controller.manager.kubeClient.qps and .burst to chart
  • adds chart tests
  • fixes chart's handling of .Values.controller.manager.extraEnv values

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 change exposes the operator's Kubernetes client QPS and Burst
configuration via cli arguments and environment variables, as well as
explicitly via chart values.

- adds `--kube-client-qps` and `--kube-client-burst` cli args
- adds `VSO_KUBE_CLIENT_QPS` and `VSO_KUBE_CLIENT_BURST` env var support
- adds `.Values.controller.manager.kubeClient.qps` and `.burst` to chart
- adds chart tests
- fixes chart's handling of `.Values.controller.manager.extraEnv` values
@hyperbolist hyperbolist requested a review from a team as a code owner January 8, 2025 00:00
Copy link

hashicorp-cla-app bot commented Jan 8, 2025

CLA assistant check
All committers have signed the CLA.

@benashz benashz added this to the v0.10.0 milestone Jan 8, 2025
@benashz benashz added the enhancement New feature or request label Jan 8, 2025
@benashz benashz self-assigned this Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants