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

[action] [PR:12913] Fix ro disk test case generate garbled syslog and break loganalyzer issue. #12929

Merged
merged 1 commit into from
May 22, 2024

Conversation

mssonicbld
Copy link
Collaborator

Fix ro disk test case generate garbled syslog and break loganalyzer issue.

Why I did it

Log rotate during ro disk may cause syslog file contains garbled characters.
These characters will break loganalyzer, to fix this issue, rotate again to cleanup syslog file.

Work item tracking
  • Microsoft ADO: 28149282

How I did it

log rotate again after ro disk test case finish,

How to verify it

Pass all test case.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Fix ro disk test case generate garbled syslog and break loganalyzer issue.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Fix ro disk test case generate garbled syslog and break loganalyzer issue.

#### Why I did it
Log rotate during ro disk may cause syslog file contains garbled characters.
These characters will break loganalyzer, to fix this issue, rotate again to cleanup syslog file.

#### How I did it
log rotate again after ro disk test case finish,

#### How to verify it
Pass all test case.

#### Description for the changelog
Fix ro disk test case generate garbled syslog and break loganalyzer issue.
@mssonicbld
Copy link
Collaborator Author

Original PR: #12913

@mssonicbld mssonicbld merged commit fccf609 into sonic-net:202305 May 22, 2024
12 checks passed
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.

2 participants