Project

General

Profile

Bug #5766

gateways widget: Incorrect monitoring of v6 over v4 tunnels

Added by Steve Beaver over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Gateway monitoring
Target version:
Start date:
01/13/2016
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.3
Affected Architecture:

Associated revisions

Revision 471e7c3a (diff)
Added by Jim Pingle over 3 years ago

Prevent GIF/GRE from making dummy/unusable gateways that show up for monitoring/routing/etc. Fixes #5766

History

#1 Updated by Steve Beaver over 3 years ago

  • File support.jpg added

#2 Updated by Steve Beaver over 3 years ago

  • Category changed from Web Interface to Gateway monitoring
  • Assignee changed from Steve Beaver to Renato Botelho

#3 Updated by Michael Kellogg over 3 years ago

in this example isn't monitoring HENETIPV6_TUNNELV6 good enough and just hide V4?

#4 Updated by Jim Pingle over 3 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)

#5 Updated by Jim Thompson over 3 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...)

#6 Updated by Jim Pingle over 3 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.

#7 Updated by Jim Pingle over 3 years ago

Here is a quick fix that works to eliminate the bum entry on GIF/GRE: https://gist.github.com/jim-p/a1b92a97e4a9ee7e30fb

#8 Updated by Jim Pingle over 3 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

#9 Updated by Renato Botelho over 3 years ago

  • Assignee changed from Renato Botelho to Jim Pingle

Works fine here

#10 Updated by Jim Pingle over 3 years ago

  • Status changed from Feedback to Resolved

Working as expected now. Closing.

#11 Updated by Jim Thompson over 3 years ago

  • File deleted (support.jpg)

Also available in: Atom PDF