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
Consider only plugin subpath when cleaning up mounts #112607
Conversation
@mattcary: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
d1f91de
to
1be20d1
Compare
func HasMountRefs(mountPath string, mountRefs []string) bool { | ||
// A mountPath typically is like | ||
// /var/lib/kubelet/plugins/kubernetes.io/some-plugin/mounts/volume-XXXX | ||
// Mount refs can look like |
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.
Is this comment saying that a path like this is something we do want to count? From the logic below, we won't count this right?
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.
Yes. I clarified the comment (at least to my understanding).
d2f416e
to
166cb4a
Compare
Change-Id: Id28c2d1c8e996c10e4ad19a993ebbe75b142c1af
166cb4a
to
3993c23
Compare
/hold I need to test this on a GKE sandbox before submitting to double-check it fixes the ephemeral storage issue. |
/cancel hold I tested this manually and it fixes the problem with a local-ssd mounted kubelet on an ubuntu image \o/ /retest (the unittest failure seems to be a flake, it passes locally) |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mattcary, msau42 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 |
/hold cancel |
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
Cherry pick #112607 Make mount ref search more robust
Cherry pick #112607 Make mount ref search more robust
Cherry pick #112607 Make mount ref search more robust
/kind bug
Fixes #112606
/assign @msau42
/sig storage