diff options
author | Donald Sharp <sharpd@nvidia.com> | 2022-10-04 13:28:51 +0200 |
---|---|---|
committer | Donald Sharp <sharpd@nvidia.com> | 2022-10-04 13:28:51 +0200 |
commit | 15d133c97e5700fd4e4e01602706799e6d439ff3 (patch) | |
tree | b2d38c898026e5a8d8b5c4df285e4b376738b7ea /isisd/fabricd.c | |
parent | Merge pull request #12043 from donaldsharp/bgp_hate (diff) | |
download | frr-15d133c97e5700fd4e4e01602706799e6d439ff3.tar.xz frr-15d133c97e5700fd4e4e01602706799e6d439ff3.zip |
fabricd: Turn off excessive logging when peering will not come up
When fabricd is configured to use an interface and there will be
no peers out that interface, the log file is filling up with:
Oct 04 10:50:03 host2 fabricd[1444769]: [HHXDJ-1DA93] ISIS-Adj (1): Threeway state change Initializing to Up
Oct 04 10:50:03 host2 fabricd[1444769]: [R18GA-MS9R7] OpenFabric: Started initial synchronization with 1111.1111.1111 on enp1s0f1np1
Oct 04 10:50:06 host2 fabricd[1444769]: [HHXDJ-1DA93] ISIS-Adj (1): Threeway state change Up to Initializing
Oct 04 10:50:07 host2 fabricd[1444769]: [NT6J7-1RYRF] OpenFabric: Initial synchronization on enp1s0f1np1 timed out!
Oct 04 10:50:07 host2 fabricd[1444769]: [R18GA-MS9R7] OpenFabric: Started initial synchronization with 3333.3333.3333 on enp1s0f0np0
Oct 04 10:50:08 host2 fabricd[1444769]: [HHXDJ-1DA93] ISIS-Adj (1): Threeway state change Up to Initializing
Oct 04 10:50:11 host2 fabricd[1444769]: [NT6J7-1RYRF] OpenFabric: Initial synchronization on enp1s0f0np0 timed out!
Oct 04 10:50:11 host2 fabricd[1444769]: [HHXDJ-1DA93] ISIS-Adj (1): Threeway state change Initializing to Up
Oct 04 10:50:11 host2 fabricd[1444769]: [R18GA-MS9R7] OpenFabric: Started initial synchronization with 1111.1111.1111 on enp1s0f1np1
Oct 04 10:50:14 host2 fabricd[1444769]: [HHXDJ-1DA93] ISIS-Adj (1): Threeway state change Up to Initializing
Oct 04 10:50:15 host2 fabricd[1444769]: [NT6J7-1RYRF] OpenFabric: Initial synchronization on enp1s0f1np1 timed out!
Oct 04 10:50:16 host2 fabricd[1444769]: [R18GA-MS9R7] OpenFabric: Started initial synchronization with 1111.1111.1111 on enp1s0f1np1
Oct 04 10:50:18 host2 fabricd[1444769]: [HHXDJ-1DA93] ISIS-Adj (1): Threeway state change Initializing to Up
The `Threeway state change..` message is guarded by a debug, but the other 2 are not.
Let's guard those with debugs since the log will be filled up rather quickly
with any sort of aggressive timers.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to '')
-rw-r--r-- | isisd/fabricd.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/isisd/fabricd.c b/isisd/fabricd.c index a37cda1ce..be035b623 100644 --- a/isisd/fabricd.c +++ b/isisd/fabricd.c @@ -254,8 +254,10 @@ static void fabricd_initial_sync_timeout(struct thread *thread) { struct fabricd *f = THREAD_ARG(thread); - zlog_info("OpenFabric: Initial synchronization on %s timed out!", - f->initial_sync_circuit->interface->name); + if (IS_DEBUG_ADJ_PACKETS) + zlog_debug( + "OpenFabric: Initial synchronization on %s timed out!", + f->initial_sync_circuit->interface->name); f->initial_sync_state = FABRICD_SYNC_PENDING; f->initial_sync_circuit = NULL; } @@ -282,9 +284,11 @@ void fabricd_initial_sync_hello(struct isis_circuit *circuit) timeout, &f->initial_sync_timeout); f->initial_sync_start = monotime(NULL); - zlog_info("OpenFabric: Started initial synchronization with %s on %s", - sysid_print(circuit->u.p2p.neighbor->sysid), - circuit->interface->name); + if (IS_DEBUG_ADJ_PACKETS) + zlog_debug( + "OpenFabric: Started initial synchronization with %s on %s", + sysid_print(circuit->u.p2p.neighbor->sysid), + circuit->interface->name); } bool fabricd_initial_sync_is_in_progress(struct isis_area *area) |