Skip to content

feat(kruise): add restConfigQPS, restConfigBurst and image.pullPolicy to manager values#167

Open
Uburro wants to merge 1 commit intoopenkruise:masterfrom
Uburro:SCHED-298/0
Open

feat(kruise): add restConfigQPS, restConfigBurst and image.pullPolicy to manager values#167
Uburro wants to merge 1 commit intoopenkruise:masterfrom
Uburro:SCHED-298/0

Conversation

@Uburro
Copy link

@Uburro Uburro commented Feb 26, 2026

Summary

Add manager.restConfigQPS, manager.restConfigBurst, and manager.image.pullPolicy
as configurable Helm values for kruise-manager.

Previously, REST client QPS/burst limits and image pull policy were hardcoded in the
chart templates and could not be overridden without patching the chart directly.

Changes:

  • Add manager.restConfigQPS (default: 30) — maps to --rest-config-qps flag
  • Add manager.restConfigBurst (default: 50) — maps to --rest-config-burst flag
  • Add manager.image.pullPolicy (default: Always) — replaces hardcoded imagePullPolicy: Always
  • Update README.md to document all new parameters

Upstream reference: https://github.com/openkruise/kruise/blob/25caf571230c581d76b27ccc782c5331cb917c2c/main.go#L76-L77

Affected versions: next, 1.9.0-alpha.4

Checklist

  • I have bumped the chart version according to versioning
  • I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • Any new values are backwards compatible and/or have sensible default.
  • I have signed off all my commits as required by DCO.

@kruise-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign fillzpp for approval by writing /assign @fillzpp in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kruise-bot
Copy link

Welcome @Uburro! It looks like this is your first PR to openkruise/charts 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants