Skip to content
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

Add a note to point out to new users that B2D bind-mounts are special-ish #9491

Merged

Conversation

SvenDowideit
Copy link
Contributor

to help with some user confusion mentioned in #4023

@fredlf @jamtur01

> (OSX) / `C:/Users` (Windows) directory - and so will only allow you to host mount
> using `docker run -v /Users/path:/ContainerDir ...` (OSX) or
> `docker run -v /c/Users/path:/ContainerDir ...` (Windows). All other paths would
> be internal to the Boot2Docker virtual machine.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

paths are internal?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i can't think of another way to note that if you do -v /etc:/banana, from OSX, then the /etc comes from the inner vm's fs, not from the outer OSX host fs. - suggestions?

@jamtur01
Copy link
Contributor

jamtur01 commented Dec 4, 2014

LGTM minor comments.

@justinmayer
Copy link

As @bradfitz said, it's not clear to me how this "closes" #4023. Seems more prudent to leave that open until his FUSE filesystem work (or an alternative solution) has been shipped.

@jessfraz
Copy link
Contributor

jessfraz commented Dec 4, 2014

But FUSE is soooo buggy

On Wed, Dec 3, 2014 at 6:10 PM, Justin Mayer notifications@github.com
wrote:

As @bradfitz https://github.com/bradfitz said
#4023 (comment),
it's not clear to me how this "closes" #4023
#4023. Seems more prudent to
leave that open until his FUSE filesystem work (or an alternative solution)
has been shipped.


Reply to this email directly or view it on GitHub
#9491 (comment).

@bradfitz
Copy link

bradfitz commented Dec 4, 2014

@jfrazelle, actually it's programs using the FUSE interface more often than not that are buggy and slow (often due to being written in C with no concurrency story), not the FUSE code in Linux itself. OS X FUSE has a less great track record, but that isn't even relevant here, because this FUSE only runs inside Linux in the Boot2Docker VM. The host machine (Windows, Mac, etc) never runs any FUSE part.

@jessfraz
Copy link
Contributor

jessfraz commented Dec 4, 2014

Ah ok cool, my experience with FUSE has been purely OS X based in the past, looking over the branch you linked, seems pretty straight forward, I do agree this shouldn't close the issue

@SvenDowideit
Copy link
Contributor Author

if my marking this pr as closing the other prompts some more work on fuse and remote volumes, then YAY :)

@SvenDowideit SvenDowideit force-pushed the note-that-volumes-in-b2d-are-special branch from 69fe1ce to 0d7c604 Compare December 4, 2014 02:51

> **Note:**
> If you are using Boot2Docker, your Docker daemon only has limited access to
> your OSX/Windows host directory. Boot2Docker tries to auto share your `/Users`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

auto-share

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete "only"

@fredlf
Copy link
Contributor

fredlf commented Dec 9, 2014

@SvenDowideit A couple of nits and a bit that needs some revision.

…-ish

Docker-DCO-1.1-Signed-off-by: Sven Dowideit <SvenDowideit@docker.com> (github: SvenDowideit)

Signed-off-by: Sven Dowideit <SvenDowideit@docker.com>
@SvenDowideit SvenDowideit force-pushed the note-that-volumes-in-b2d-are-special branch from 0d7c604 to e529885 Compare December 30, 2014 03:38
@SvenDowideit
Copy link
Contributor Author

@fredlf is that better?

@fredlf
Copy link
Contributor

fredlf commented Dec 31, 2014

Yes, much better. I still think there's part of a thought missing between the first and second sentence, but it's good enough to publish. Thanks @SvenDowideit

fredlf pushed a commit that referenced this pull request Dec 31, 2014
…re-special

Add a note to point out to new users that B2D bind-mounts are special-ish
@fredlf fredlf merged commit 880aeab into moby:master Dec 31, 2014
SvenDowideit pushed a commit to SvenDowideit/docker that referenced this pull request Jan 27, 2015
…2d-are-special

Add a note to point out to new users that B2D bind-mounts are special-ish
(cherry picked from commit 880aeab)

Docker-DCO-1.1-Signed-off-by: Fred Lifton <fred.lifton@docker.com> (github: SvenDowideit)
@SvenDowideit SvenDowideit deleted the note-that-volumes-in-b2d-are-special branch January 28, 2015 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants