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

[201911][show] show logging CLI support for logs stored in tmpfs #3132

Open
wants to merge 1 commit into
base: 201911
Choose a base branch
from

Conversation

mihirpat1
Copy link
Contributor

@mihirpat1 mihirpat1 commented Jan 24, 2024

Signed-off-by: Mihir Patel patelmi@microsoft.com

Backport of #2641

ADO - 16810429

What I did

Enabling "show logging" CLI to show logs from tmpfs if it exists.

How I did it

Added a check in the CLI handler to see if the path "/var/log.tmpfs" exists and display the logs from this directory if it exists. In case if the path doesn't exist, logs will be displayed from "/var/log"

How to verify it

Following tests were performed:

  1. "show logging" o/p when syslog resides in "/var/log"
  2. "show logging -f" o/p when syslog resides in "/var/log"
  3. "show logging" o/p when syslog resides in "/var/log.tmpfs"
  4. "show logging -f" o/p when syslog resides in "/var/log.tmpfs"

For detailed test results, please refer to Unit-test_logging_cli.txt

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant