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

Vendor libnetwork v0.7.0-dev.7 : Experimental MacVlan and IPVlan network drivers #21122

Merged
merged 3 commits into from
Mar 13, 2016

Conversation

mavenugo
Copy link
Contributor

This awesome contribution comes from @nerdalert via moby/libnetwork#964.
This PR also includes the required documentation to get up & running with these drivers.

Thanks to the CNM APIs, this is purely new driver additions and no changes to the libnetwork core or engine core.

We decided to accept it as experimental to get user feedback and make any improvements.

Note : Though some of the distros backported IPVlan drivers on the 3.19 kernel, it is extremely buggy (like this one : https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1475434) and we decided to keep the base support for IPVlan driver as 4.2.
MacVlan driver is fairly stable from 3.9,

@mavenugo mavenugo changed the title Experimental Macvlan and IPVlan network drivers Vendor libnetwork v0.7.0-dev.6 : Experimental Macvlan and IPVlan network drivers Mar 11, 2016
@mavenugo mavenugo changed the title Vendor libnetwork v0.7.0-dev.6 : Experimental Macvlan and IPVlan network drivers Vendor libnetwork v0.7.0-dev.6 : Experimental MacVlan and IPVlan network drivers Mar 11, 2016
@cpuguy83
Copy link
Member

I think the docs for ipvlan and macvlan should go into /experimental instead of the main docs until we move this over to stable.


- To check your current kernel version, use `uname -r` to display your kernel version
- Macvlan Linux kernel v3.9–3.19 and 4.0+
- Ipvlan Linux kernel v4.0+ (earlier versions do support Ipvlan but)
Copy link
Member

Choose a reason for hiding this comment

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

but what? (should say something like support for earlier kernels exists but is buggy)

@thaJeztah
Copy link
Member

Documentation failure looks genuine;

frontmatter: (engine/userguide/networking/vlan-networks.md) Unexpected non-whitespace char: # Macvlan and Ipvlan Docker Network Drivers

@@ -0,0 +1,685 @@


# Macvlan and Ipvlan Docker Network Drivers
Copy link
Member

Choose a reason for hiding this comment

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

Possibly a tab instead of a space somewhere?

@thaJeztah
Copy link
Member

oh, and optimized versions of those images (saved off 41% of the file size);
21122-images.zip

@thaJeztah thaJeztah added the status/failing-ci Indicates that the PR in its current state fails the test suite label Mar 11, 2016
@cpuguy83
Copy link
Member

Just the couple of issues.

@mavenugo mavenugo force-pushed the mviv branch 2 times, most recently from 0e05a26 to 05b56c4 Compare March 11, 2016 20:36
* Experimental support for macvlan and ipvlan drivers

Signed-off-by: Madhu Venugopal <madhu@docker.com>
@mavenugo mavenugo changed the title Vendor libnetwork v0.7.0-dev.6 : Experimental MacVlan and IPVlan network drivers Vendor libnetwork v0.7.0-dev.7 : Experimental MacVlan and IPVlan network drivers Mar 11, 2016
Signed-off-by: Brent Salisbury <brent@docker.com>
Signed-off-by: Madhu Venugopal <madhu@docker.com>
Signed-off-by: Brent Salisbury <brent@docker.com>
@mavenugo
Copy link
Contributor Author

@cpuguy83 @thaJeztah we resolved the comments. PTAL.

@thaJeztah thaJeztah removed the status/failing-ci Indicates that the PR in its current state fails the test suite label Mar 12, 2016
@thaJeztah thaJeztah added this to the 1.11.0 milestone Mar 12, 2016
@thaJeztah
Copy link
Member

Ping @calavera @tiborvass if you have some time to review as well, hopefully we can get this merged soon, as it's needed for other changes coming up

@icecrime
Copy link
Contributor

Very cool, and strictly adding (experimental) drivers to libnetwork, so LGTM! 🎉

@cpuguy83
Copy link
Member

Tested out these drivers. Really awesome! We should iterate on some of the options/when to error out etc. But I'm good with this as is for experimental.

LGTM.

@cpuguy83
Copy link
Member

I see we've still got docs in the user guide, so I'll defer to the docs team for this.
Usually with experimental we don't add them to the main docs.. but again will defer to docs team.

@cpuguy83
Copy link
Member

ping @thaJeztah

@mavenugo
Copy link
Contributor Author

The docs currently is not linked anywhere. This is added purely as an user-guide for folks to give it a try directly from the docker/docker master repo and not from docs.docker.com.

@thaJeztah
Copy link
Member

Away from home right now, but feel free to merge, I can have a look at docs after that and fix up later; I know this PR is important, so don't let me uphold it

On 13 Mar 2016, at 19:13, Madhu Venugopal notifications@github.com wrote:

The docs currently is not linked anywhere. This is added purely as an user-guide for folks to give it a try directly from the docker/docker master repo and not from docs.docker.com.


Reply to this email directly or view it on GitHub.

@LK4D4
Copy link
Contributor

LK4D4 commented Mar 13, 2016

@thaJeztah Yeah, let's merge and try. Sounds like exciting stuff.

LK4D4 added a commit that referenced this pull request Mar 13, 2016
Vendor libnetwork v0.7.0-dev.7 : Experimental MacVlan and IPVlan network drivers
@LK4D4 LK4D4 merged commit 2d74f78 into moby:master Mar 13, 2016
@mavenugo
Copy link
Contributor Author

Thanks @cpuguy83 @icecrime @LK4D4 @thaJeztah

With the addition of these 2 new drivers, Ops have complete control over the docker networking stack including seamless underlay plumbing.

Thanks @nerdalert for the contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants