-
Notifications
You must be signed in to change notification settings - Fork 38.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
Migrate kubelet to ConfigMapManager interface and use TTL-based caching manager #46470
Migrate kubelet to ConfigMapManager interface and use TTL-based caching manager #46470
Conversation
2425f55
to
ccd7600
Compare
@shyamjvs please fix the test flake. It's due to your modification of
|
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.
Just minor nit - also please fix tests.
pkg/kubelet/pod/pod_manager.go
Outdated
@@ -163,6 +166,10 @@ func (pm *basicManager) updatePodsInternal(pods ...*v1.Pod) { | |||
// not register pod, as it doesn't really matter. | |||
pm.secretManager.RegisterPod(pod) | |||
} | |||
if pm.configMapManager != nil { | |||
// TODO: Same as above. |
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.
Please copy a TODO instead of writing "Same as above"
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.
Done.
ccd7600
to
817a8e2
Compare
LGTM @dchen1107 @yujuhong - can you of you please take a look? |
817a8e2
to
80d137a
Compare
@k8s-bot pull-kubernetes-unit test this |
80d137a
to
94707c7
Compare
@shyamjvs - please rebase |
1fb3708
to
1cf6b33
Compare
@k8s-bot pull-kubernetes-kubemark-e2e-gce test this
|
Applying lgtm based on @wojtek-t 's #46470 (comment) above. |
@k8s-bot pull-kubernetes-verify test this |
@k8s-bot pull-kubernetes-e2e-gce-etcd3 test this |
1 similar comment
@k8s-bot pull-kubernetes-e2e-gce-etcd3 test this |
@k8s-bot pull-kubernetes-e2e-gce-etcd3 test this |
Added 'retest-not-required' label as it happened for the 3rd time that all PR tests passed when I rerun them, but 'pull-kubernetes-e2e-gce-etcd3' (which has been super-flaky recently) fails during the premerge round, making me repeat the cycle all over again. |
@k8s-bot pull-kubernetes-e2e-gce-etcd3 test this |
looks like etcd3 job is bumping into #46713 |
@spiffxp Thanks for the lead. Will wait for the new run's result (after bumping up subnet quota in the project) and then retry. |
@k8s-bot pull-kubernetes-e2e-gce-etcd3 test this |
1 similar comment
@k8s-bot pull-kubernetes-e2e-gce-etcd3 test this |
Automatic merge from submit-queue |
Fixes #41379
Sometime ago we moved to a secret manager interface for kubelet to manage secrets.
This PR's first commit moves config map management also to a similar interface.
The second commit adds TTL-based CachingConfigMapManager (similar to CachingSecretManager) and makes kubelet use it.
/cc @kubernetes/sig-node-pr-reviews @kubernetes/sig-scalability-misc @wojtek-t @dchen1107