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

Module doesn't support initcwnd or initrwnd #29

Open
shawn-sterling opened this issue Jan 5, 2014 · 0 comments
Open

Module doesn't support initcwnd or initrwnd #29

shawn-sterling opened this issue Jan 5, 2014 · 0 comments

Comments

@shawn-sterling
Copy link

Hello,

Currently the module won't support setting a initcwnd or initrwnd. In order to set these to be persistent, one needs to

  1. Remove any gateway statements from /etc/sysconfig/network
  2. Remove any gateway statements from any ifcfg-$interface files
  3. put a default statement into the route-$interface file. IE:
default via 10.1.1.1 dev bond0 proto static initcwnd 10 initrwnd 10

*NOTE: initcwnd requires $rhel 6, initrwnd requires $rhel 6.4+

The module is currently using the route-$interface file for setting up static routes (which is the correct place for them), but no option to setup a default route statement with other settings like the initcwnd/initrwnd.

This could be solved with using concat fragments, so the 1st fragment could be the default gateway and the 2nd be the static routes.

Or the network::route define could be changed to not take the $interface from $name, so one could have several network::route defines (the way it is now, with the file declared in the define, will break if you have multiple). This would still use concat but would allow for many concats.

Or something else entirely.

Either way it's quite a bit of change to the module and adds a dependency which may not be desired.

I'm happy to take a stab at any of the above and send a pull request, but would like to hear your opinion first.

Have a good day.

-Shawn

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

No branches or pull requests

1 participant