-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Set leader-elect=false for scheduler and controller manager #8431
Conversation
both the scheduler and controller manager constantly ping the apiserver to ensure leader election. From the docs: ``` Enable this when running replicated components for high availability. ``` which we don't really need in minikube. This should reduce overall overhead by ~12%
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: priyawadhwa The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Travis tests have failedHey @priyawadhwa, 1st Buildmake test
TravisBuddy Request Identifier: f4e34170-aa93-11ea-9064-4d6590cbb359 |
Codecov Report
@@ Coverage Diff @@
## master #8431 +/- ##
=======================================
Coverage 34.05% 34.05%
=======================================
Files 153 153
Lines 9840 9840
=======================================
Hits 3351 3351
Misses 6086 6086
Partials 403 403 |
/ok-to-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems easy enough! Do you mind adding it to the other ktmpl versions for consistency among Kubernetes releases?
kvm2 Driver |
kvm2 Driver Times for Minikube (PR 8431): [81.18499008799999 63.627436309000004 64.317619203] Averages Time Per Log
docker Driver Times for Minikube (PR 8431): [25.500323742 26.082818228 28.064057732000006] Averages Time Per Log
|
Both the scheduler and controller manager constantly ping the apiserver to ensure leader election. From the docs:
which we don't really need in minikube. This should reduce overall overhead by ~12% according to Thomas's calculations, and around 14% according to mine. Let's play it safe and say 12%, which gets us halfway to finishing #5682