start should support --mount-string argument

[trentza]

FEATURE REQUEST

Environment:

Minikube version: v0.30.0

• OS: MacOS Darwin Kernel Version 18.2.0
• VM Driver: hyperkit
• ISO version: minikube-v0.30.0.iso

We have a well scripted process for spinning up our stack, currently it uses NFS shares.
We are looking to refactor this to use the minikube start --mount --mount-string "/blah:/blah"

We are struggling to get write access to the mount-point via this method.

We have tested with minikube mount with the --9p-version=9p2000.L and this gives us write access.

During minikube start we would like to make use of the --9p-version flag as per the minikube mount.

seems some similar discussion was taking place on this issue: #2290

[narg95]

Hi,
Same here, any answer or workaround to get --9p-version=9p2000.L on minikube start?

[shaneknapp]

yes pls!

[GrigorievNick]

In version 0.35 is see log
- Creating mount C:\\Users\\myhome\\IdeaProjects\\bigkube\\target\\scala-2.11:/data/jars ...
but when i use minikube ssh;ls /data/jars i get no such folder.
thats same issue? or i do something wrong

[shaneknapp]

bump! this would be extremely useful, especially for the ephemeral minikube clusters i have to spin up for the apache spark k8s integration tests. the only way i can get the PV to mount correctly is with a separate minikube mount command after the cluster starts, and do some bash gymnastics to keep track of the PID so i can ensure that the mount is gone when the integration tests are done.

our build script looks kinda like this:

minikube --vm-driver=kvm2 start --memory 6000 --cpus 8
minikube mount ${WORKSPACE}/tmp:/tmp/spark-k8s-integration-tests --9p-version=9p2000.L --gid=0 --uid=185 &
MOUNT_PID=$(jobs -rp)

...tests run...

kill -9 $MOUNT_PID
minikube stop

being able to pass in the additional mount args (gid, 9p-version, etc) directly through the minikube start call would be great, awesome, superb and much, much less fragile than doing things this way.

[tstromberg]

@shaneknapp - Agreed. I'd happily review any PR which makes the above happen.

[shaneknapp]

@shaneknapp - Agreed. I'd happily review any PR which makes the above happen.

I SEE WHAT YOU DID THERE!

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[shaneknapp]

/remove-lifecycle stale

[nanikjava]

@tstromberg interested to tackle this issue. Would you be able to point me in the right direction to start.

So far I can see that the mount command code is inside mount.go but at high level what does the new code required to do ?

Thanks

[nanikjava]

/assign

[no3am]

I'll just add our use case for traction I suppose as we need this fix urgently.
I'm running postgres on minikube, I don't want a fresh entity, I've took a ready data from production and wanna mount it to the postgres pod running on minikube - I'm able to do that !

Because the data is stored on my Mac and mounted into the pod, it has root:root on the data folder which is making it impossible for me do anything with it after mounting it, of course the postgres can't load it properly as well, I'd need to mount the folder with 999:999 or something like that.

Seems if the above issue is fixed, my issue will be fixed as well, in the meanwhile I'm looking for ways to bypass it as we're blocked :(

[sharifelgamal]

@nanikjava still interested in working on this?

[nanikjava]

@nanikjava still interested in working on this?

Going to unassign myself for now, as trying to work on another ticket.

[Divya063]

/assign

[Divya063]

I started working on this issue, but the folder is not getting mounted after running command - minikube start --mount --mount-string "~/test:/test" #2481 (comment)

[zamazan4ik]

Any updates?

[prezha]

@Divya063, @zamazan4ik i cannot reproduce the issue on the latest version (1.17.1 atm) with docker - ie, this worked for me:

minikube start --mount --mount-string ~/tmp:/tmp/app

[prezha]

/close

[k8s-ci-robot]

@prezha: You can't close an active issue/PR unless you authored it or you are a collaborator.

In response to this:

/close

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.

[lacarvalho91]

--mount-string seems to work to the point that you can mount a directory, but it still doesn't seem to support passing extra mount flags e.g. --uid

[spowelljr]

@lacarvalho91 you are correct that we don't support passing --uid on start, this is something that can be added to the start flags. However, currently you can use minikube mount --uid on an already started minikube cluster.

[lacarvalho91]

@spowelljr yeah I'm doing that now, would be awesome to be able to do it on start though!

[shaneknapp]

yep, agree -- it'd be great if we could pass these args in @ startup...

…

On Wed, Oct 27, 2021 at 12:29 PM Lawrence Carvalho ***@***.***> wrote: @spowelljr <https://github.com/spowelljr> yeah I'm doing that now, would be awesome to be able to do it on start though! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3327 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAMIHLEFJFVXMGR54TXBUATUJBOI7ANCNFSM4GDGMXNQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[spowelljr]

I created an issue to track this request, I might have time to implement this next month and be included with minikube v1.25 release.

#12802