sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
194 stars 714 forks source link

Add loganalyzer_common_ignore regexs for FEC counters on broadcom #13408

Closed byu343 closed 2 months ago

byu343 commented 3 months ago

Description of PR

Summary: This change is to ignore some error in syslog for reading certain FEC counters on unsupported port. Broadcom is in the process of making some fix to it, and before the fix, we could use this change to ignore the error and pass the tests that will use config reload.

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

StormLiangMS commented 3 months ago

hi @byu343 could you help to open an issue and update the issue id in the PR so we won't forget to uncomment this line when the fix is ready?

StormLiangMS commented 3 months ago

/azp run Azure.sonic-mgmt

azure-pipelines[bot] commented 3 months ago
Azure Pipelines successfully started running 1 pipeline(s).
byu343 commented 2 months ago

This changes looks not needed for the latest SAI.