Skip to content

Commit 7f21c0b

Browse files
authored
[FRR] remove the whole block of outchannel properly (#6045)
- Why I did it Fix issue #6043 - How I did it We are disabling in container frr log. The log entries are sent to base image and are logged in /var/log/quagga/bgpd.log. However, we need to remove the whole outchannel config block to avoid an error message raised by rsyslogd. - How to verify it Without the change, test_autorestart bgp container will fail on loganalyer errors. With the change, restarting bgp container is no longer generating error message and the test will pass. The log generated by frr continued appearing in /var/log/quagga/bgpd.log
1 parent 00d3cb7 commit 7f21c0b

File tree

1 file changed

+24
-7
lines changed

1 file changed

+24
-7
lines changed

src/sonic-frr/patch/0007-frr-remove-frr-log-outchannel-to-var-log-frr.log.patch

+24-7
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
From 1f150992b074d07bce5157d203319512a3aeb4dc Mon Sep 17 00:00:00 2001
1+
From 31464e3fd0f677bc15286e99d4a30e2e33480c53 Mon Sep 17 00:00:00 2001
22
From: Ying Xie <[email protected]>
3-
Date: Wed, 18 Nov 2020 19:24:51 +0000
3+
Date: Wed, 25 Nov 2020 22:28:43 +0000
44
Subject: [PATCH] [frr] remove frr log outchannel to /var/log/frr.log
55

66
SONiC runs frr inside a docker and the logs are sent to base image
@@ -10,19 +10,36 @@ space eventually.
1010

1111
Signed-off-by: Ying Xie <[email protected]>
1212
---
13-
tools/etc/rsyslog.d/45-frr.conf | 2 +-
14-
1 file changed, 1 insertion(+), 1 deletion(-)
13+
tools/etc/rsyslog.d/45-frr.conf | 19 -------------------
14+
1 file changed, 19 deletions(-)
1515

1616
diff --git a/tools/etc/rsyslog.d/45-frr.conf b/tools/etc/rsyslog.d/45-frr.conf
17-
index ff7cd4831..af3c2c109 100644
17+
index feeeb13f1..fcd23d23a 100644
1818
--- a/tools/etc/rsyslog.d/45-frr.conf
1919
+++ b/tools/etc/rsyslog.d/45-frr.conf
20-
@@ -2,7 +2,7 @@
20+
@@ -2,25 +2,6 @@
2121
# to /var/log/frr/frr.log, then drops the message so it does
2222
# not also go to /var/log/syslog, so the messages are not duplicated
2323

2424
-$outchannel frr_log,/var/log/frr/frr.log
25-
+$outchannel frr_log
25+
-if $programname == 'babeld' or
26+
- $programname == 'bgpd' or
27+
- $programname == 'eigrpd' or
28+
- $programname == 'frr' or
29+
- $programname == 'isisd' or
30+
- $programname == 'fabricd' or
31+
- $programname == 'ldpd' or
32+
- $programname == 'nhrpd' or
33+
- $programname == 'ospf6d' or
34+
- $programname == 'ospfd' or
35+
- $programname == 'pimd' or
36+
- $programname == 'ripd' or
37+
- $programname == 'ripngd' or
38+
- $programname == 'vrrpd' or
39+
- $programname == 'watchfrr' or
40+
- $programname == 'zebra'
41+
- then :omfile:$frr_log
42+
-
2643
if $programname == 'babeld' or
2744
$programname == 'bgpd' or
2845
$programname == 'eigrpd' or

0 commit comments

Comments
 (0)