Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[frr] remove FRR rsyslog file outchannel #5962

Merged
merged 1 commit into from
Nov 21, 2020
Merged

Conversation

yxieca
Copy link
Contributor

@yxieca yxieca commented Nov 18, 2020

- Why I did it
frr is creating /var/log/frr/frr.log inside the frr docker and letting it grow. It will eventually exhaust hard drive space.

To fixe issue #5965

- How I did it
Remove rsyslog file outchannel so that frr won't generate /var/log/frr/frr.log inside the docker.

- How to verify it
Manually removed the outchannel and restart BGP docker, making sure that /var/log/frr/frr.log is no longer created inside the docker.

While restarting bgp docker, observed that base image /var/log/quagga/bgpd.log continued to grow and captured all FRR logs.

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

Signed-off-by: Ying Xie [email protected]

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]>
@yxieca yxieca changed the title [frr] mount /var/log/frr onto frr docker and configure logrotate [frr] remove FRR rsyslog file outchannel Nov 18, 2020
@jleveque
Copy link
Contributor

jleveque commented Nov 18, 2020

This may be suited for a future PR, but now that we are using FRR instead of quagga, the path /var/log/quagga/ is a bit misleading. Maybe we should consider /var/log/frr/ or a more generic /var/log/bgp/.

@rlhui
Copy link
Contributor

rlhui commented Nov 18, 2020

@yxieca - would you like to file an issue? Thanks.

@gechiang
Copy link
Collaborator

@yxieca There is a dependency on this FRR log file for the functionality of bgpmon that is running inside the BGP docker.
If any changes made to either remove this log file or renamed it or moved to somewhere else, the code indie bgpmon will also need to be changed so that is can still take advantage of knowing there are FRR activities by monitoring this file's timestamp in order to avoid unnecessary polling neighbor info from BGP...

@gechiang gechiang self-requested a review November 18, 2020 22:37
@yxieca
Copy link
Contributor Author

yxieca commented Nov 19, 2020

Closing this PR due to bgpmon issue Gen-Hwa identified. Putting back the original change in #5966

@yxieca yxieca closed this Nov 19, 2020
@yxieca yxieca deleted the frr branch November 19, 2020 01:05
@yxieca yxieca restored the frr branch November 19, 2020 22:10
@yxieca yxieca reopened this Nov 19, 2020
@lguohan
Copy link
Collaborator

lguohan commented Nov 20, 2020

i am confused on which approach we are taking now?

@yxieca
Copy link
Contributor Author

yxieca commented Nov 20, 2020

i am confused on which approach we are taking now?

According to Gen-Hwa/Judy/Arvind, we are taking this approach because bgpmon can work without tapping into the log. Judy will make a follow up change to make sure multi-ASIC frr logs are captured and put into /var/log/quagga/bgpd.log with name space tag.

@yxieca yxieca merged commit d3c1a5b into sonic-net:master Nov 21, 2020
@yxieca yxieca deleted the frr branch November 21, 2020 03:36
abdosi pushed a commit that referenced this pull request Nov 21, 2020
- Why I did it
frr is creating /var/log/frr/frr.log inside the frr docker and letting it grow. It will eventually exhaust hard drive space.

To fixe issue #5965

- How I did it
Remove rsyslog file outchannel so that frr won't generate /var/log/frr/frr.log inside the docker.

- How to verify it
Manually removed the outchannel and restart BGP docker, making sure that /var/log/frr/frr.log is no longer created inside the docker.

While restarting bgp docker, observed that base image /var/log/quagga/bgpd.log continued to grow and captured all FRR logs.
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
- Why I did it
frr is creating /var/log/frr/frr.log inside the frr docker and letting it grow. It will eventually exhaust hard drive space.

To fixe issue sonic-net#5965

- How I did it
Remove rsyslog file outchannel so that frr won't generate /var/log/frr/frr.log inside the docker.

- How to verify it
Manually removed the outchannel and restart BGP docker, making sure that /var/log/frr/frr.log is no longer created inside the docker.

While restarting bgp docker, observed that base image /var/log/quagga/bgpd.log continued to grow and captured all FRR logs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants