snc
snc copied to clipboard
crc-org
Add cifs-utils package to hyperV bundle
this package provides utilities for managing mounts of CIFS network file systems
its needed to make use of windows default CIFS/SMB feature for network file sharing between hosts
I've downloaded the bundle from CI and can use it for testing, we should wait to merge this until we decide if we really want to use CIFS/SMB, see https://github.com/code-ready/crc/issues/3230#issuecomment-1211686987
/hold
I am not convinced CIFS is currently the priority. I would say sshfs is more generic.
/hold
Till we finalize we should go with cifs for windows file share work.
I think we can go ahead with using CIFS/SMB for file sharing on windows, opened https://github.com/code-ready/crc/pull/3308
I am not convinced CIFS is currently the priority. I would say sshfs is more generic.
the reason to choose this over sshfs was mainly because of performance, while doing a very basic copy test using the SMB file sharing, it had good performance as mentioned in https://github.com/code-ready/crc/issues/3230#issuecomment-1211686987 also WSL and docker uses CIFS for their file sharing
although sshfs is generic and will also work on macOS and linux, we currently have working file sharing with virtiofs on these platforms which is more performant.
WSL
WSL2 does not use CIFS from host to VM (for this they use 9p). They however use CIFS to mount VM to host using \\wsl\ (this however is a kernel based solution)
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gbraad, praveenkumar
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [gbraad,praveenkumar]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
![openshift-ci[bot] avatar](https://avatars.githubusercontent.com/in/91280?v=4 "Published by a pub.dev verified publisher"){kind=small}
@praveenkumar: once the present PR merges, I will cherry-pick it on top of master in a new PR and assign it to you.
In response to this:
/cherry-pick master
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.
@praveenkumar: new pull request created: #583
In response to this:
/cherry-pick master
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.