-
Notifications
You must be signed in to change notification settings - Fork 321
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
Parse ipv6 addresses in ipvs backend status #40
Comments
Interesting, what kernel version are you using? The current code parses IPv6 via the same hex format. |
@matthiasr, where did you find the sample of a |
I don't remember I'm afraid. I hope I configured this on some machine and this is what I got … |
Actually, reading prometheus/node_exporter#67 I don't think I did. I had no IPv6 example to work off and it probably never worked this way. |
Paging @juliusv who, if I remember correctly, had a hand or two in the IPv6 support for IPVS :) |
@SuperQ these machines run a mix of |
@matthiasr I didnt touch this procfs file back then, but I added a Generic Netlink interface to read and write IPVS configs (which was needed because the old sockopt-based interface couldn't accommodate v6 addresses). You can take a look at what the Netlink interface looks like today, maybe it can give you the same answers: https://github.com/torvalds/linux/blob/ca78d3173cff3503bcd15723b049757f75762d15/net/netfilter/ipvs/ip_vs_ctl.c#L2836 |
Hokay, I have no idea what how to read those 😆 If the |
SGTM |
Confirmed working, thanks a lot @matthiasr ! I've opened prometheus/node_exporter#576 to add this to |
Add /proc/pressure parser (closes prometheus#40)
Fix IPVS ipv6 parsing, see also prometheus/procfs#40
We are running
ipvs
collector fromnode_exporter
on a machine receiving ipv6 traffic too, thoughnode_exporter
0.13 fails with this:And the
/proc/net/ip_vs
file looks like this (after redacting the addresses)The text was updated successfully, but these errors were encountered: