Skip to content
This repository has been archived by the owner on May 14, 2019. It is now read-only.

Added DVR option to neutron #203

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added DVR option to neutron #203

wants to merge 2 commits into from

Conversation

rrmichel
Copy link

Enabled DVR support for neutron.

Default is disabled.

@mmailand
Copy link

Hi,
I have have a few points to this patch.

  1. Shouldn't we make it configurable via the hiera yaml file?
  2. I think DVR needs the l2-population und arp-responder plugin as prerequisite.

We have already done a bit of work in this direction, should we collaborate?

What is the best way to do that? Just a pull request with our changes?

@rrmichel
Copy link
Author

@mmailand It is already configurable via hiera but i missed the example file.

l2-population and arp-responder sure?

Any other changes you made in your "fork" ?

@mmailand
Copy link

@rrmichel Yes I am sure that you need l2-population and arp-responder for the DVR mode.
No, no other changes in my fork.

@cmurphy
Copy link
Contributor

cmurphy commented Jul 15, 2015

@mmailand, @rrmichel, is there agreement on which set of pull requests should be merged?

@mmailand
Copy link

@cmurphy There was no further communication on this topic. I think my patch series is more complete. I could make a new branch with the three topic-branches each one commit in the new branch. That will make the merge more easier for you.

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

Successfully merging this pull request may close these issues.

3 participants