]> git.puffer.fish Git - mirror/frr.git/commit
tools: make --quiet actually suppress output 14418/head
authorJonas Gorski <jonas.gorski@bisdn.de>
Thu, 14 Sep 2023 15:04:16 +0000 (17:04 +0200)
committerJonas Gorski <jonas.gorski@bisdn.de>
Thu, 14 Sep 2023 15:20:25 +0000 (17:20 +0200)
commit312d5ee1592f8c5b616d330233d1de2643f759e2
tree3b55832391e2f63548191e6cd41689ce4577ea93
parent75dbd45c5570c088dc0e2cf83e1b22b0ec0a30b3
tools: make --quiet actually suppress output

When calling daemon_stop() with --quiet and e.g. the pidfile is empty,
it won't return early since while "$fail" is set, "$2" is "--quiet", so
the if condition isn't met and it will continue executing, resulting
in error messages in the log:

> Sep 14 14:48:33 localhost watchfrr[2085]: [YFT0P-5Q5YX] Forked background command [pid 2086]: /usr/lib/frr/watchfrr.sh restart all
> Sep 14 14:48:33 localhost frrinit.sh[2075]: /usr/lib/frr/frrcommon.sh: line 216: kill: `': not a pid or valid job spec
> Sep 14 14:48:33 localhost frrinit.sh[2075]: /usr/lib/frr/frrcommon.sh: line 216: kill: `': not a pid or valid job spec
> Sep 14 14:48:33 localhost frrinit.sh[2075]: /usr/lib/frr/frrcommon.sh: line 216: kill: `': not a pid or valid job spec

Fix this by moving the --quiet check into the block to log_failure_msg(),
and also add the check to all other invocations of log_*_msg() to make
--quiet properly suppress output.

Fixes: 19a99d89f088 ("tools: suppress unuseful warnings during restarting frr")
Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
tools/frrcommon.sh.in