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

docker network create -d bridge test works if there is already an overlay network named test? #23983

Closed
allencloud opened this issue Jun 27, 2016 · 3 comments · Fixed by #24158
Labels
area/networking priority/P3 Best effort: those are nice to have / minor issues. version/1.12
Milestone

Comments

@allencloud
Copy link
Contributor

Output of docker version:

root@ubuntu:~# docker version
Client:
 Version:      1.12.0-rc2
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   906eacd
 Built:        Fri Jun 17 21:02:41 2016
 OS/Arch:      linux/amd64
 Experimental: true

Server:
 Version:      1.12.0-rc2
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   906eacd
 Built:        Fri Jun 17 21:02:41 2016
 OS/Arch:      linux/amd64
 Experimental: true

Output of docker info:

root@ubuntu:~# docker info
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 39
Server Version: 1.12.0-rc2
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 380
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: null overlay bridge host
Swarm: active
 NodeID: 4b1et52uyr6nquyj7js30fgcu
 IsManager: Yes
 Managers: 1
 Nodes: 3
 CACertHash: sha256:9ebab8ad017aa10bd8eee00ba34f7e22932deeee38adf6e864786fb643668caf
Runtimes: default
Default Runtime: default
Security Options: apparmor seccomp
Kernel Version: 3.19.0-25-generic
Operating System: Ubuntu 14.04.3 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.954 GiB
Name: ubuntu
ID: HRBI:EV6M:3NAC:CXZ3:BDS7:UOKL:PD6Y:ZCK4:V3PF:V7MR:CU2Y:3LGG
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Experimental: true
Insecure Registries:
 127.0.0.0/8

Additional environment details (AWS, VirtualBox, physical, etc.):

Steps to reproduce the issue:
1.docker network create -d overlay test to create an overlay network named test
2.docker network create -d bridge test to create a bridge network named test

Above steps can work. Actually they own the same name. It is a CONFLICT in the network naming.

While if I create bridge network named test, then create overlay network named test, daemon will report an error named:

Error response from daemon: network with name test already exists

In addition, This bug will result in docker network inspect test works if two networks with same name on a single daemon.

@cpuguy83
Copy link
Member

ping @mrjana

@tiborvass tiborvass added priority/P2 Normal priority: default priority applied. priority/P3 Best effort: those are nice to have / minor issues. and removed priority/P2 Normal priority: default priority applied. labels Jun 27, 2016
@mrjana
Copy link
Contributor

mrjana commented Jun 27, 2016

ping @mavenugo

@mavenugo
Copy link
Contributor

thanks for reporting. I will address this along with other related issues.

mavenugo added a commit to mavenugo/docker that referenced this issue Jul 7, 2016
Signed-off-by: Madhu Venugopal <madhu@docker.com>
tiborvass pushed a commit to tiborvass/docker that referenced this issue Jul 8, 2016
Signed-off-by: Madhu Venugopal <madhu@docker.com>
(cherry picked from commit 6a4b21b)
Signed-off-by: Tibor Vass <tibor@docker.com>
rchicoli pushed a commit to rchicoli/docker that referenced this issue Nov 12, 2016
Signed-off-by: Madhu Venugopal <madhu@docker.com>
(cherry picked from commit 6a4b21b)
Signed-off-by: Tibor Vass <tibor@docker.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/networking priority/P3 Best effort: those are nice to have / minor issues. version/1.12
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants