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.

Sign up for GitHub

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

Jump to bottom

docker network create default driver #17500

Closed
vieux opened this issue Oct 29, 2015 · 4 comments
Closed

docker network create default driver #17500

vieux opened this issue Oct 29, 2015 · 4 comments
Assignees
@calavera
Labels
kind/bug Bugs are bugs. The cause may or may not be known at triage time so debugging may be needed.
Milestone
1.9.0

Comments

@vieux
Copy link
Contributor

vieux commented Oct 29, 2015

Before #16910

The docker cli sent "" as driver, so engine was using bridge and swarm was using overlay if -d wasn't specified.

After #16910

The docker cli always send "bridge" as driver, so it makes it impossible for swarm to know if a user specified "bride" or if it's the default.

I'd like to revert this behaviour.
@vieux
Copy link
Contributor Author

vieux commented Oct 29, 2015

ping @mavenugo @mrjana @dave-tucker

@tiborvass tiborvass added this to the 1.9.0 milestone Oct 29, 2015
@tiborvass
Copy link
Contributor

whoever sends PR to fix this, please add a test.

@tiborvass tiborvass added the kind/bug Bugs are bugs. The cause may or may not be known at triage time so debugging may be needed. label Oct 29, 2015
@calavera
Copy link
Contributor

working on it.

@calavera calavera self-assigned this Oct 29, 2015
@calavera calavera mentioned this issue Oct 29, 2015
Merged
@calavera
Copy link
Contributor

Take a look at #17504.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@calavera calavera

Labels
kind/bug Bugs are bugs. The cause may or may not be known at triage time so debugging may be needed.
Projects
None yet
Milestone
1.9.0
Development

No branches or pull requests
3 participants
@calavera @tiborvass @vieux