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 back max concurrent reconciles environment variable #210

Merged
merged 2 commits into from
Sep 22, 2020

Conversation

JohnStarich
Copy link
Member

Includes and fixes #205

I previously thought this underlying controller feature had been removed in kubebuilder v2. Glad I was mistaken! This PR adds it back, pulling on the initial work by @qibobo from #205.

The new option is surfaced in the deployment.yaml for easy editing.

@codecov
Copy link

codecov bot commented Sep 22, 2020

Codecov Report

Merging #210 into master will increase coverage by 0.05%.
The diff coverage is 62.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
+ Coverage   91.05%   91.11%   +0.05%     
==========================================
  Files           6        6              
  Lines         749      754       +5     
==========================================
+ Hits          682      687       +5     
  Misses         44       44              
  Partials       23       23              
Impacted Files Coverage Δ
controllers/manager_setup.go 87.23% <40.00%> (+0.56%) ⬆️
controllers/binding_controller.go 87.02% <100.00%> (+0.03%) ⬆️
controllers/service_controller.go 96.32% <100.00%> (+0.01%) ⬆️
controllers/token_controller.go 98.14% <100.00%> (+0.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3388d60...36f3617. Read the comment docs.

@JohnStarich JohnStarich merged commit cb323dd into master Sep 22, 2020
@JohnStarich JohnStarich deleted the feature/max-concurrent-reconciles branch September 22, 2020 02:37
artberger pushed a commit that referenced this pull request Sep 23, 2020
* MaxConcurrentReconciles can be set through env var
* Combine max reconciles into main Config struct, surface it in deployment.yaml

Co-authored-by: qibobo <[email protected]>
Signed-off-by: Art Berger <[email protected]>
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