diff options
| author | lynne <lynne@voltanet.io> | 2021-06-11 09:53:23 -0400 |
|---|---|---|
| committer | lynne <lynne@voltanet.io> | 2021-06-14 15:46:49 -0400 |
| commit | 1b1f7b4f1540a276e4ec6fda3e08c78a9ca5982c (patch) | |
| tree | 9449a60b65d89e5cebe50fbb1c2e771e2fd2e780 /ospf6d/ospf6_interface.c | |
| parent | 352cc841d897eb343f12d7f85c7f346ab3c29beb (diff) | |
ospf6d: move error logs out from behind debug flags
The logging in ospf6 is very verbose. If you turn on logging on a scaled
system you get too many logs. The problem is that there are some errors
that occur that are hidden behind the debug flags, and to see these errors
we currently need to turn on the debug logging. This change converts these
error logs to warnings and removes the debug flags.
Signed-off-by: Lynne Morrison <lynne@voltanet.io>
Diffstat (limited to 'ospf6d/ospf6_interface.c')
| -rw-r--r-- | ospf6d/ospf6_interface.c | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/ospf6d/ospf6_interface.c b/ospf6d/ospf6_interface.c index 952c7f8b27..6763e7457f 100644 --- a/ospf6d/ospf6_interface.c +++ b/ospf6d/ospf6_interface.c @@ -701,20 +701,17 @@ int interface_up(struct thread *thread) /* check physical interface is up */ if (!if_is_operative(oi->interface)) { - if (IS_OSPF6_DEBUG_INTERFACE) - zlog_debug( - "Interface %s is down, can't execute [InterfaceUp]", - oi->interface->name); + zlog_warn("Interface %s is down, can't execute [InterfaceUp]", + oi->interface->name); return 0; } /* check interface has a link-local address */ if (!(ospf6_interface_get_linklocal_address(oi->interface) || if_is_loopback_or_vrf(oi->interface))) { - if (IS_OSPF6_DEBUG_INTERFACE) - zlog_debug( - "Interface %s has no link local address, can't execute [InterfaceUp]", - oi->interface->name); + zlog_warn( + "Interface %s has no link local address, can't execute [InterfaceUp]", + oi->interface->name); return 0; } @@ -731,7 +728,7 @@ int interface_up(struct thread *thread) /* If no area assigned, return */ if (oi->area == NULL) { - zlog_debug( + zlog_warn( "%s: Not scheduleing Hello for %s as there is no area assigned yet", __func__, oi->interface->name); return 0; |