Skip to content

Libnetwork to program container interface's MAC#803

Merged
mavenugo merged 1 commit intomoby:masterfrom
aboch:pm
Mar 4, 2016
Merged

Libnetwork to program container interface's MAC#803
mavenugo merged 1 commit intomoby:masterfrom
aboch:pm

Conversation

@aboch
Copy link
Copy Markdown
Contributor

@aboch aboch commented Dec 5, 2015

  • Consistently with what it does for IP addresses, libnetwork
    will also program the container interface's MAC address with
    the value set by network driver in InterfaceInfo.

Signed-off-by: Alessandro Boch aboch@docker.com

- Consistently with what it does for IP addresses, libnetwork
  will also program the container interface's MAC address with
  the value set by network driver in InterfaceInfo.

Signed-off-by: Alessandro Boch <aboch@docker.com>
@aboch aboch changed the title Libnetwork to set container interface's MAC Libnetwork to program container interface's MAC Dec 8, 2015
Comment thread osl/interface_linux.go
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I guess this function should be added to other platforms ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

My understanding is that so far only the osl functions which get invoked from libnetwork are ported to the respective platform package, as this is needed to allow building libnetwork on the other platforms.

This function is not being called outside of the osl package.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes only osl.Sandbox interface methods need to be implemented or stubbed for other platforms. Other platforms may not even have a nwIface

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It is not just this function. but we need to stub a bunch of other existing interface methods that are not implemented.
We can handle it in another PR.

@mavenugo
Copy link
Copy Markdown
Contributor

mavenugo commented Mar 4, 2016

LGTM.

mavenugo added a commit that referenced this pull request Mar 4, 2016
Libnetwork to program container interface's MAC
@mavenugo mavenugo merged commit 77c1c58 into moby:master Mar 4, 2016
@aboch aboch deleted the pm branch March 7, 2016 16:37
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.

3 participants