Skip to content

Fix controller workqueue rate limits, resource limits, and deletion log noise - #710

Merged
kraney merged 4 commits into
openconfig:mainfrom
kraney:ratelimit2
Aug 28, 2026
Merged

Fix controller workqueue rate limits, resource limits, and deletion log noise#710
kraney merged 4 commits into
openconfig:mainfrom
kraney:ratelimit2

Conversation

@kraney

@kraney kraney commented Aug 20, 2026

Copy link
Copy Markdown
Contributor
  • Align controller workqueue RateLimiter with REST client QPS (100 QPS, 200 burst) in Options, resolving the default 10 QPS WorkQueue bottleneck.
  • Increase controller manager deployment resource limits (CPU: 2, Memory: 1Gi) to prevent OOM kills and CPU throttling under high concurrency and scale.
  • Check for apierrors.IsNotFound(err) before logging errors in Reconcile(), eliminating noisy ERROR stack traces during resource deletion/cleanup.

…og noise

- Align controller workqueue RateLimiter with REST client QPS (100 QPS, 200 burst) in Options, resolving the default 10 QPS WorkQueue bottleneck.
- Increase controller manager deployment resource limits (CPU: 2, Memory: 1Gi) to prevent OOM kills and CPU throttling under high concurrency and scale.
- Check for apierrors.IsNotFound(err) before logging errors in Reconcile(), eliminating noisy ERROR stack traces during resource deletion/cleanup.
@kraney
kraney requested a review from a team as a code owner August 20, 2026 22:57
@bstoll

bstoll commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

/gcbrun

@bstoll

bstoll commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

/gcbrun

@kraney
kraney merged commit de0cccc into openconfig:main Aug 28, 2026
15 checks passed
@kraney
kraney deleted the ratelimit2 branch August 28, 2026 20:26
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.

2 participants