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

Support VXLAN GBP #455

Merged
merged 2 commits into from
Jul 12, 2016
Merged

Support VXLAN GBP #455

merged 2 commits into from
Jul 12, 2016

Conversation

evanfarrar
Copy link
Contributor

Linux versions 4.0 and above include Group Based Policy support in VXLAN. This patch provides a GBP config option for the flannel VXLAN backend and bumps the vendored netlink library to include a fix for GBP support.

References
- VXLAN GBP RFC
- GBP support in Linux 4+

cc: @rosenhouse @dbellotti @jaydunk

@evanfarrar evanfarrar closed this Jun 29, 2016
@evanfarrar evanfarrar reopened this Jun 29, 2016
@evanfarrar evanfarrar closed this Jun 29, 2016
@evanfarrar evanfarrar reopened this Jun 29, 2016
@rosenhouse
Copy link

Hey there @tomdee and @steveej, mind taking a look? We're adding this feature to cloudfoundry and would love to get the changes merged upstream. Thanks!

@steveej
Copy link
Contributor

steveej commented Jul 12, 2016

Please adjust the commit message(s) to conform to our CONTRIBUTING guidelines, thanks a lot!

evanfarrar and others added 2 commits July 12, 2016 01:05
recent netlink includes fix for VXLAN GBP
(to re-generate glide.lock, run `glide up -s -v -u`)
With this change, the vxlan backend may be configured to create GBP-enabled
vxlan devices.
@rosenhouse
Copy link

@steveej Fixed up commit messages and rebased on master.

@steveej
Copy link
Contributor

steveej commented Jul 12, 2016

@rosenhouse didn't know you have two identities ;-) I'll merge this as is but keep label for the PR to indicate it needs testing at one point.

@steveej steveej merged commit d69940b into flannel-io:master Jul 12, 2016
@rosenhouse rosenhouse deleted the vxlan-gbp-2 branch July 13, 2016 04:12
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.

3 participants