Skip to content

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

Merged
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12913
May 22, 2024
Merged

[action] [PR:12913] Fix ro disk test case generate garbled syslog and break loganalyzer issue.#12930
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12913

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 a2b6c4e into sonic-net:202311 May 22, 2024
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