Lrouhier/update_readme - Githubissues

lrouhier commented 5 years ago

Updates the README following discussion about issue #13 Add information about mounting local folder on docker container if it is not done automatically.

lrouhier commented 5 years ago

Not sure if it is the right place to add this paragraph. on the one hand the user should have already run the docker container, on the other if he has no problem it takes some space ...

YangHee-Min commented 5 years ago

Not sure if it is the right place to add this paragraph. on the one hand the user should have already run the docker container, on the other if he has no problem it takes some space ...

@lrouhier Sorry not quite sure I understood what the problem was. Which paragraph are we talking about in this case?

lrouhier commented 5 years ago

Not sure if it is the right place to add this paragraph. on the one hand the user should have already run the docker container, on the other if he has no problem it takes some space ...

@lrouhier Sorry not quite sure I understood what the problem was. Which paragraph are we talking about in this case?

The one I added.

YangHee-Min commented 5 years ago

After following the steps (since my docker_shared_folder wasn't highlighted in green), I still could not manage to get the docker_shared_folder to be highlighted in green. I gave USERS all permissions within C:/Users/ folder, yet cannot make a directory inside it with command mkdir test.

Is there something I must reset on my computer due to yesterday's tests?

jcohenadad commented 5 years ago

@YangHee-Min can you try to re-create another folder (different name)?

lrouhier commented 5 years ago

Yesterday i opened à New quickstart terminal after performing the ssh and I kept both open. I thought it wasn't needed as you're performing ssh on the machine and running the container on top of it.

YangHee-Min commented 5 years ago

@YangHee-Min can you try to re-create another folder (different name)?

@jcohenadad I get message: mkdir: cannot create directory 'new_test': Permission denied

jcohenadad commented 5 years ago

@YangHee-Min can you try to re-create another folder (different name)?

@jcohenadad I get message: mkdir: cannot create directory 'new_test': Permission denied

i meant: from windows (not your container), as explained here

YangHee-Min commented 5 years ago

@YangHee-Min can you try to re-create another folder (different name)?

@jcohenadad I get message: mkdir: cannot create directory 'new_test': Permission denied

i meant: from windows (not your container), as explained here

Yes I was able to make the folder

jcohenadad commented 5 years ago

@YangHee-Min can you try to re-create another folder (different name)?

@jcohenadad I get message: mkdir: cannot create directory 'new_test': Permission denied

i meant: from windows (not your container), as explained here

Yes I was able to make the folder

i know you were able to make the folder, my point was to re-test your folder sharing problem with another folder (to rule out the possibility that by playing with permissions on the same folder for the past few days you introduced some conflicts).

YangHee-Min commented 5 years ago

@jcohenadad @lrouhier I retested and everything works!

jcohenadad commented 5 years ago

@jcohenadad @lrouhier I retested and everything works!

do you know what went wrong during the previous test? in case a user has the same problem, we know what to tell our users...

YangHee-Min commented 5 years ago

@jcohenadad @lrouhier I retested and everything works!

do you know what went wrong during the previous test? in case a user has the same problem, we know what to tell our users...

I hadn't properly mounted the shared folder before running docker run. In other words, I did not follow the instructions properly

spinalcordtoolbox / sct_docker

Lrouhier/update_readme #34