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
Find a better place to initialize volume manager informers #96316
Comments
@kubernetes/sig-storage-feature-requests |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
/remove-lifecycle stale |
/help-wanted |
This issue has not been updated in over 1 year, and should be re-triaged. You can:
For more details on the triage process, see https://www.kubernetes.dev/docs/guide/issue-triage/ /remove-triage accepted |
/triage accepted |
This issue has not been updated in over 1 year, and should be re-triaged. You can:
For more details on the triage process, see https://www.kubernetes.dev/docs/guide/issue-triage/ /remove-triage accepted |
What would you like to be added:
Background: #93130 (comment)
Right now
WaitForCacheSync
for csi informers is called right before we need to read from it, which is well after any initialization, and gets called repeatedly.However, there isn't a great place to move it right now. Places that have been tried:
volumePluginMgr.Run
. Problem here is that that the volume reconciler has already started before this, which means csi could already be processing requests.volumePluginMgr.Run
before starting the volume reconciler. But the volume reconciler is needed to bootstrap kubelet before there's a connection to apiserver.csiPlugin.Init
. This appears to have similar bootstrap ordering issues.Maybe we need some sort of
plugin.Start
interface?The text was updated successfully, but these errors were encountered: