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
Pass additional flags to subpath mount to avoid flakes in certain conditions #104340
Pass additional flags to subpath mount to avoid flakes in certain conditions #104340
Conversation
7faa5dc
to
eac4824
Compare
/retest |
@@ -0,0 +1,278 @@ | |||
// +build linux |
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.
We synced offline and potentially found a way to extend the mount interface without needing to copy the original implementations.
…ath/subpath_* history.
- The kubelet startup code will create a new Mounter instance that wraps the existing k8s.io/utils/mount instance - Provided implementations for all the platforms
- Also adding an implementation for all the platforms
…lume/util/subpath/subpath_fake_mounter.go history.
hey @Random-Liu, I'd really appreciate a review on this PR, in summary the kubelet subpather is receiving a new mounter that wraps k8s.io/utils/mount |
/cc @Random-Liu |
/approve for the cmd/kubelet change. |
hey @kubernetes/release-managers, could you please review this PR? thanks in advance! |
/sig windows |
looks like this was not cherry-picked using the process |
/triage accepted from parent PR: |
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.
from RelEng:
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cpanato, mauriciopoppe, msau42, yujuhong 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 |
…ional-mount-flag-new-mounter-119 Pass additional flags to subpath mount to avoid flakes in certain conditions
Merge pull request kubernetes#104340 from mauriciopoppe/subpath-additional-moun…
What type of PR is this?
/kind bug
/sig storage
What this PR does / why we need it:
Pass additional flags to subpath mount to avoid flakes in certain conditions, supersedes #104254, the new strategy is to create another mount implementation only for subpath usage.
MountSensitiveWithMountFlags
that will be used only for subpath mount purposesMountSensitiveWithMountFlags
in subpath_linux.go:doBindSubpath()Also I created a GCE cluster and verified it in the kubelet logs
Does this PR introduce a user-facing change?
/cc @msau42 @jingxu97 @jsafrane