Skip to content

Commit

Permalink
[frr] remove frr rsyslog file outchannel
Browse files Browse the repository at this point in the history
SONiC runs FRR inside a docker, the logs are captured in base
image. Leaving file outchannel inside docker will generate log
file inside the docker and take harddrive space unnecessarily.

Signed-off-by: Ying Xie <[email protected]>
  • Loading branch information
yxieca committed Nov 18, 2020
1 parent b595a6e commit ae670b7
Show file tree
Hide file tree
Showing 2 changed files with 32 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
From 1f150992b074d07bce5157d203319512a3aeb4dc Mon Sep 17 00:00:00 2001
From: Ying Xie <[email protected]>
Date: Wed, 18 Nov 2020 19:24:51 +0000
Subject: [PATCH] [frr] remove frr log outchannel to /var/log/frr.log

SONiC runs frr inside a docker and the logs are sent to base image
via rsyslog and recorded already. There is no need to keep the
frr.log inside the docker. It will grow and take all harddrive
space eventually.

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

diff --git a/tools/etc/rsyslog.d/45-frr.conf b/tools/etc/rsyslog.d/45-frr.conf
index ff7cd4831..af3c2c109 100644
--- a/tools/etc/rsyslog.d/45-frr.conf
+++ b/tools/etc/rsyslog.d/45-frr.conf
@@ -2,7 +2,7 @@
# to /var/log/frr/frr.log, then drops the message so it does
# not also go to /var/log/syslog, so the messages are not duplicated

-$outchannel frr_log,/var/log/frr/frr.log
+$outchannel frr_log
if $programname == 'babeld' or
$programname == 'bgpd' or
$programname == 'eigrpd' or
--
2.17.1

1 change: 1 addition & 0 deletions src/sonic-frr/patch/series
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,4 @@
0004-Allow-BGP-attr-NEXT_HOP-to-be-0.0.0.0-due-to-allevia.patch
0005-nexthops-compare-vrf-only-if-ip-type.patch
0006-changes-for-making-snmp-socket-non-blocking.patch
0007-frr-remove-frr-log-outchannel-to-var-log-frr.log.patch

0 comments on commit ae670b7

Please sign in to comment.