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

v6uplink #60

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

v6uplink #60

wants to merge 7 commits into from

Conversation

ohrensessel
Copy link
Contributor

allows to set up a v6 uplink connection via a bgp session

@sargon
Copy link
Contributor

sargon commented Dec 14, 2014

Since only bird6 resources are affected some module name with ffnord::bird6 prefix would be much nicer. If you don't want to enlarge the bird6.pp you could create a directory bird6 and put new files in there. Don't forget to update the README for new functionality.

Apart from above, this looks okay for me.

@ohrensessel
Copy link
Contributor Author

I'm working on functionality to also set up a GRE tunnel for the v6 uplink, which will then not really fit into the bird6 module. so i think it is reasonable to have this as a separate module.

@sargon
Copy link
Contributor

sargon commented Dec 15, 2014

Okay, I see, next long term issue is then to get bird6 related stuff back into the bird6 type, maybe build some wrapper types. I personally find it unsettling to have bird configuration snipped spread all over the place. Better to keep the main configuration logic in one place, but as told above, later.

@ohrensessel
Copy link
Contributor Author

for the moment an uplink device should be added to the mesh forward chain, as it is assumed that the server providing the uplink is trusted and also has an appropriate firewall in operation.

@ohrensessel
Copy link
Contributor Author

Should be nearly finished. Probably only missing corrected firewall rules so that connection tracking is not hindering asymmetric routing

@MTRNord
Copy link
Contributor

MTRNord commented Jan 17, 2017

Info Kommentar: Meine PR ( #176 )hatte dadurch, dass ich die lint fehler berichtigt habe auch zeilen verschoben. Deswegen der conflict.

@rubo77 rubo77 force-pushed the master branch 2 times, most recently from 70b4764 to 6d8bf60 Compare May 29, 2017 20:47
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