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

Fuzziness in source/manual/how-tos/lan_bridge.rst #285

Closed
spiriapbergeron opened this issue Sep 27, 2020 · 2 comments
Closed

Fuzziness in source/manual/how-tos/lan_bridge.rst #285

spiriapbergeron opened this issue Sep 27, 2020 · 2 comments
Assignees
Labels
cleanup Low impact changes

Comments

@spiriapbergeron
Copy link

Hi. First, thanks a million for this howto. It saved a ton of time and screwing around.

I do see this readme being used by the official opnsense documentation also!
https://wiki.opnsense.org/manual/how-tos/lan_bridge.html

In general the instructions are quite clear, but the last step I'm unsure about.

For Step 6, you state:

Go to System->Settings->Tunables and select using the pen button net.link.bridge.pfil_member and set the value to 0.

and

Select the tunable net.link.bridge.pfil_bridge and set the value to 1

But on my current opnsense version, these two variables do not exist.
So, if the variables don't exist, should we add them (I did).

So I guess the fuzziness is around the word "select" or "create if they don't exist" or something.

Thanks for the howto!

@AdSchellevis
Copy link
Member

we do ship it by default, but I don't mind adding a small comment inline as well.
https://github.com/opnsense/core/blob/0b379917aecaa3cad23d1fae9514997563d63cee/src/etc/config.xml.sample#L112-L116

@AdSchellevis AdSchellevis self-assigned this Sep 27, 2020
@AdSchellevis AdSchellevis added the cleanup Low impact changes label Sep 27, 2020
@spiriapbergeron
Copy link
Author

I had never heard of these variables until I read your howto -- and I can say for certain I didn't delete them somehow, and they definitely weren't part of my installation. Very weird.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Low impact changes
Development

No branches or pull requests

2 participants