Skip both empty line and also new line in the output of portstat.#7657
Skip both empty line and also new line in the output of portstat.#7657wenyiz2021 merged 1 commit intosonic-net:masterfrom
Conversation
|
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
|
Add @arlakshm @wenyiz2021 for visibility. |
|
thanks @ysmanman for this fix! |
Description of PR
#7560 was closed with #7537. However, we still observed the the issue in our testing even with #7537. The fix in #7537 is incomplete because it only skips new line, but fails to skip empty line. Fix the issue in this PR to skip both new line and empty line.
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
How did you do it?
How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation