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.

Sign up for GitHub

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

Jump to bottom

Set the number of concurrent reconciles for the Runtime controller to 25 #475

Merged
merged 18 commits into from
Nov 25, 2024
Merged

Set the number of concurrent reconciles for the Runtime controller to 25 #475

merged 18 commits into from
Nov 25, 2024

Conversation

koala7659
Copy link
Contributor

@koala7659 koala7659 commented Oct 31, 2024 •
edited
Loading

  • 25 workers for Runtime Controller
  • 25 workers for GardenCluster Controller
  • Reading Shoot kubeconfing from secret without using Gardener API
  • increased requeue times for different Gardener operation (shoot create/update/delete)
  • Configurable Gardener client parameters - Timeout, rateLimiterQPS, rateLimiterBurst

@koala7659 koala7659 requested a review from a team as a code owner October 31, 2024 17:08
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 31, 2024
@koala7659 koala7659 changed the title Set the max number of concurrent reconciles for the runtime controller to 10 KIM - Set the max number of concurrent reconciles for the Runtime controller to 10 Oct 31, 2024
@koala7659 koala7659 changed the title KIM - Set the max number of concurrent reconciles for the Runtime controller to 10 Set the max number of concurrent reconciles for the Runtime controller to 10 Oct 31, 2024
@koala7659 koala7659 added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 31, 2024
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 31, 2024
@Disper Disper changed the title Set the max number of concurrent reconciles for the Runtime controller to 10 WIP Set the max number of concurrent reconciles for the Runtime controller to 10 Nov 4, 2024
@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 4, 2024
@koala7659 koala7659 self-assigned this Nov 5, 2024
@kyma-bot kyma-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 18, 2024
@koala7659 koala7659 force-pushed the kim-add-multiple-workers branch from 5fd5420 to 986c9a1 Compare November 19, 2024 08:14
@kyma-bot kyma-bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 19, 2024
@kyma-bot kyma-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 19, 2024
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 20, 2024
@koala7659 koala7659 changed the title WIP Set the max number of concurrent reconciles for the Runtime controller to 10 Set the number of concurrent reconciles for the Runtime controller to 25 Nov 25, 2024
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 25, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 25, 2024
@kyma-bot kyma-bot merged commit 119c367 into kyma-project:main Nov 25, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tobiscr tobiscr tobiscr approved these changes

Assignees

@koala7659 koala7659

@tobiscr tobiscr

Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@koala7659 @tobiscr @kyma-bot @m00g3n