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
Change code to use staging/k8s.io/mount-utils #94527
Conversation
/assign @msau42 |
facadbd
to
fe1ed8c
Compare
TODO : We still need to remove utils/mounts from vendor. |
/test pull-kubernetes-e2e-azure-file |
/test pull-kubernetes-e2e-azure-file |
/test pull-kubernetes-e2e-azure-file-windows |
fe1ed8c
to
a48885d
Compare
There were some more commits to k8s.io/utils/mount after your initial merge. Can you copy those commits over too? cc @andyzhangx @jingxu97 for awareness. |
@msau42 all commits are moved, the last one picked up is "fix: return error with fewer mount options on Windows" from 8/15. |
/test pull-kubernetes-e2e-windows-gce |
2 similar comments
/test pull-kubernetes-e2e-windows-gce |
/test pull-kubernetes-e2e-windows-gce |
f6e5933
to
fbe5dae
Compare
/test pull-kubernetes-e2e-kind-ipv6 |
/test pull-kubernetes-e2e-gce-storage-slow |
@msau42 @andyzhangx @divyenpatel Can you please review ASAP. |
/test pull-kubernetes-e2e-windows-gce |
windows test always timeout as always.. |
/retest |
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.
Thanks!
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brahmaroutu, thockin 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 |
/milestone v1.20 |
@brahmaroutu: You must be a member of the kubernetes/milestone-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your and have them propose you as an additional delegate for this responsibility. In response to this:
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. |
@msau42 This PR may need milestone set for it to be merged? |
/milestone v1.20 |
/release-note-none |
/test pull-kubernetes-integration |
/kind cleanup
/release-note-none
sig/storage
sig/testing
What this PR does / why we need it:
This PR is moving the vendor/utils/mount package into k/k to better manage and test the source code with Storage e2e.