-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Use shared informers in scheduler and token controller #14321
Use shared informers in scheduler and token controller #14321
Conversation
pkg/cmd/server/start/start_master.go
Outdated
@@ -609,6 +609,9 @@ func startControllers(oc *origin.MasterConfig, kc *kubernetes.MasterConfig) erro | |||
// Start these first, because they provide credentials for other controllers' clients | |||
oc.RunServiceAccountsController() | |||
oc.RunServiceAccountTokensController(controllerManagerOptions) | |||
// Must start a few informers now | |||
oc.Informers.KubernetesInformers().Start(utilwait.NeverStop) |
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.
These need to start to make the SA token controller active, right? Be more descriptive in your comment. Lots of people end up here.
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.
Yeah, will add more of a comment
7ff83d5
to
a8ba30b
Compare
Evaluated for origin test up to a8ba30b |
lgtm |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/1695/) (Base Commit: 61b1f2a) |
[merge] for great performance |
Evaluated for origin merge up to a8ba30b |
continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/785/) (Base Commit: 4a682ba) |
[merge][severity: blocker] memory use is still p0 for 3.6 |
Contains upstream fixes and a few minor reaction changes. Rebase pick was a little ugly because of the number of changes, but verified it passed in upstream branch and downstream.
@deads2k @sttts [test]