Bug #5766
closedgateways widget: Incorrect monitoring of v6 over v4 tunnels
100%
Updated by Anonymous almost 9 years ago
- Category changed from Web Interface to Gateway Monitoring
- Assignee changed from Anonymous to Renato Botelho
Updated by Michael Kellogg almost 9 years ago
in this example isn't monitoring HENETIPV6_TUNNELV6 good enough and just hide V4?
Updated by Jim Pingle almost 9 years ago
That's the idea. The interface has no v4 IP so it should have no v4 gateway, yet the system makes a v4 gateway entry (that shouldn't be there)
Updated by Jim Thompson almost 9 years ago
"just hide V4" is a work-around Michael.
We're better than that.
let's make it work right.
[details deleted]
(I love having my system at home detailed in public...)
Updated by Jim Pingle almost 9 years ago
Looks like this GIF (and also GRE) case could be fixed by some better testing in source:src/etc/inc/interfaces.inc#L5573 and source:src/etc/inc/interfaces.inc#L5605 (interface_has_gateway() and interface_has_gatewayv6()). In the case of gif/gre it should check the interface address to make sure the interface has an address of the desired type. That should be a safe test since it is statically configured in the gif settings, unlike the other types that can change.
Updated by Jim Pingle almost 9 years ago
Here is a quick fix that works to eliminate the bum entry on GIF/GRE: https://gist.github.com/jim-p/a1b92a97e4a9ee7e30fb
Updated by Jim Pingle almost 9 years ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset 471e7c3ac7276568c0fea726d699fb63dd87d3eb.
Updated by Renato Botelho almost 9 years ago
- Assignee changed from Renato Botelho to Jim Pingle
Works fine here
Updated by Jim Pingle almost 9 years ago
- Status changed from Feedback to Resolved
Working as expected now. Closing.