8000 --mac-address doesn't accept a non-local MAC address · Issue #9771 · moby/moby · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

--mac-address doesn't accept a non-local MAC address #9771

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
Closed
giacomini opened this issue Dec 22, 2014 · 1 comment · Fixed by docker-archive/libcontainer#310 or #9779
Closed

--mac-address doesn't accept a non-local MAC address #9771

giacomini opened this issue Dec 22, 2014 · 1 comment · Fixed by docker-archive/libcontainer#310 or #9779

Comments

@giacomini
Copy link

I would like to be able to set the mac address of a container to a non-locally administered MAC address, e.g. 00:16:3E:08:00:50. At the moment this is forbidden:

$ docker version
Client version: 1.4.1
Client API version: 1.16
Go version (client): go1.3.3
Git commit (client): 5bc2ff8
OS/Arch (client): linux/amd64
Server version: 1.4.1
Server API version: 1.16
Go version (server): go1.3.3
Git commit (server): 5bc2ff8
$ docker run --mac-address=00:16:3E:08:00:50 ubuntu ifconfig
setup networking set eth0 mac Incorrect Local MAC Address specified: 00:16:3e:08:00:50FATA[0000] Error response from daemon: Cannot start container 307befaecce951ff7663353b1e5d8e5a16a37119c3a71d613ade623f617915fa: setup networking set eth0 mac Incorrect Local MAC Address specified: 00:16:3e:08:00:50

The check is done at https://github.com/docker/docker/blob/master/vendor/src/github.com/docker/libcontainer/netlink/netlink_linux.go#L528.
Is there any technical reason for that check? wouldn't a warning be enough?
@jessfraz
Copy link
Contributor

We could definitely have a warning instead. I can create a PR.

This was referenced Dec 22, 2014
Merged
Merged
@jessfraz jessfraz mentioned this issue Jan 8, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow non local mac-address.
Add test for non local mac address. jessfraz/docker
2 participants
@jessfraz @giacomini
0