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

Add second fastd instance with MTU 1280 #116

Closed
wants to merge 2 commits into from
Closed

Add second fastd instance with MTU 1280 #116

wants to merge 2 commits into from

Conversation

rubo77
Copy link
Contributor

@rubo77 rubo77 commented May 10, 2015

to circumvent unstable connections with Kabel Deutschland

@ohrensessel
Copy link
Contributor

Thanks for your effort. I would, however, like to see this more flexible. Now we have the possibility to support two music, but in case of a mtu change we might to need support e.g. three instances. Because of that it would be great if we would have the possibility to support multiple fastd instances (not only two), maybe by just giving a list of mtus and ports.

This also would be a great preparation for separate fastd instances for node and inter-gateway traffic.

@ohrensessel
Copy link
Contributor

Two music = two mtus. Stupid autocorrect.

@rubo77
Copy link
Contributor Author

rubo77 commented May 10, 2015

Is there anything missing in this setup? If not, you can accept this for a start and I will create another patch these days, that will replace the identifyer string "low" with a configurable string, for each instance in a list of mtus, identifyer strings and ports. So it stays downward compatible

@sargon
Copy link
Contributor

sargon commented May 10, 2015

This patch-set didn't solve the generic problem of multiple fastd instances per community, it rater introduce further special cases, which we afterwords have to clean up. I rate against a merge.

@rubo77
Copy link
Contributor Author

rubo77 commented May 10, 2015

I have to find out how to create a file for each defined block with puppet

@rubo77
Copy link
Contributor Author

rubo77 commented Jun 8, 2015

I created another aproach here: https://github.com/rubo77/ffnord-example/commits/multiple_fastd which uses the branch https://github.com/rubo77/ffnord-puppet-gateway/tree/multi_fastd

Can someone please test this if it is ok like this?

@rubo77
Copy link
Contributor Author

rubo77 commented Jun 9, 2015

new PR: #120

@rubo77 rubo77 closed this Jun 9, 2015
@rubo77 rubo77 deleted the fastd1280 branch June 12, 2015 20:40
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