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

Updating dockervolumes.md for Windows mounting #1032

Merged
merged 1 commit into from
Jan 18, 2017
Merged

Updating dockervolumes.md for Windows mounting #1032

merged 1 commit into from
Jan 18, 2017

Conversation

josephmosby
Copy link
Contributor

Proposed changes

The code snippet for mounting Windows directories won't work: it will produce an "Invalid bind mount spec" error as Docker cannot properly read the path. The core error was addressed in Docker issue #12590, this is just an update to the tutorial.

Unreleased project version (optional)

N/A

Related issues (optional)

moby/moby#12590 (comment)

The code snippet for mounting Windows directories won't work: it will produce an "Invalid bind mount spec" error as Docker cannot properly read the path. The core error was addressed in [#12590](moby/moby#12590 (comment)), this is just an update to the tutorial.
FrenchBen pushed a commit to FrenchBen/docker.github.io that referenced this pull request Jan 18, 2017
The format for node advertisement in swarm and docker engine (for
networking) is the same, and can be shared. Doing so allows us to start
toward removing the swarm-join container, and improves our HA situation
after engine discovery is run. Resolves docker#670.

Specifically, swarm was previously somewhat HA: if you set up 1
controller, 2 replicas and 1 node, then you could lose the controller
and still remain functional unless the swarm node needed to restart.
This change improves HA by removing the above failure mode as long as
engine discovery has been run and is up to date.

See also docker-archive/classicswarm#1368 which implemented this flag, and lays out the
future scenario we hope to achieve, and also docker#1032 which is our goal of
removing swarm join with automatical engine discovery.
FrenchBen pushed a commit to FrenchBen/docker.github.io that referenced this pull request Jan 18, 2017
The format for node advertisement in swarm and docker engine (for
networking) is the same, and can be shared. Doing so allows us to start
toward removing the swarm-join container, and improves our HA situation
after engine discovery is run. Resolves docker#670.

Specifically, swarm was previously somewhat HA: if you set up 1
controller, 2 replicas and 1 node, then you could lose the controller
and still remain functional unless the swarm node needed to restart.
This change improves HA by removing the above failure mode as long as
engine discovery has been run and is up to date.

See also docker-archive/classicswarm#1368 which implemented this flag, and lays out the
future scenario we hope to achieve, and also docker#1032 which is our goal of
removing swarm join with automatical engine discovery.
Copy link

@mdlinville mdlinville left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@mdlinville mdlinville merged commit 6db5538 into docker:master Jan 18, 2017
@josephmosby josephmosby deleted the patch-1 branch October 19, 2017 13:19
bermudezmt pushed a commit that referenced this pull request May 22, 2019
1013 - Fix ToC indentation, missing images
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.

2 participants