-
Notifications
You must be signed in to change notification settings - Fork 0
Add Command sonic-clear logging #1
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: mati <[email protected]>
Looks good, |
I think that one is already part of master, I'll have another look. |
|
Signed-off-by: mati <[email protected]>
Signed-off-by: mati <[email protected]>
BTW it looks like they added a test to the show logging |
Signed-off-by: mati <[email protected]>
Signed-off-by: mati <[email protected]>
Signed-off-by: mati <[email protected]>
Signed-off-by: mati <[email protected]>
What Python version is used in SONiC? |
Signed-off-by: mati <[email protected]>
Not sure if correct for all platform/builds, but VS (master) runs 3.11.2 |
Fixed Problems/New Changes: change format syntax to new form use glob only where needed Should be Tested: ~ Signed-off-by: mati <[email protected]>
Signed-off-by: mati <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
don't know if needed, don't think it's worth the effort, but if mandatory ... |
What I did
Added command
sonic-clear logging - delete syslog & syslog.1
sonic-clear logging -a/--all - delete syslog* files
How I did it
Equivalent to "show logging" command, working on files at /var/log or at /var/log.tmpfs
delete relevant syslog* files
How to verify it
Repeat 1-4 with
Clear logging --all
verify also that show logging behave correctly after clear
Previous command output (if the output of a command-line utility has changed)
NR
New command output (if the output of a command-line utility has changed)
none