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

MRTG config generation: add IPv6 enable flag #868

Open
lucix-mich opened this issue Jul 28, 2023 · 1 comment
Open

MRTG config generation: add IPv6 enable flag #868

lucix-mich opened this issue Jul 28, 2023 · 1 comment

Comments

@lucix-mich
Copy link

Hello,

MRTG does still not enable IPv6 by default; you need to add a flag to mrtg.cfg to enable polling targets over IPv6. IXP Manager doesn't contain functionality to add the IPv6 enable flag if required, based on target switches management IP(v6).

The current mrtg.cfg template ($IXPROOT/resources/views/services/grapher/mrtg/header.foil.php) does not set the IPv6 enable flag as default. Looking over mrtg's IPv6 doc (https://oss.oetiker.ch/mrtg/doc/mrtg-ipv6.en.html), it seems safe to enable IPv6 support by default.

Hence, I propose to add this line to the mrtg template:

EnableIPv6: Yes

I see this as the easier solution versus determining programmatically if the flag is required in the current context. Of course one can patch the template locally, but that change would get overwritten on upgrades.

We are currently patching the generated mrtg config file, but that's also a hacky solution.

Thanks for considering this FR!

@nickhilliard
Copy link
Member

definitely not a bad idea, but looking through the mrtg code + running a quick test, this enables a pile of internal code path changes and directly uses import Socket6, which redefines several low-level primitives including sub sockaddr_in6. It also changes how snmp calls are handled.

This needs to be tested before we can commit, as there's a chance it could break existing configs.

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

No branches or pull requests

2 participants