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

[LogAnalyzer] Add ignore log for MACSEC counter #6224

Merged
merged 3 commits into from
Aug 25, 2022

Conversation

jimmyzhai
Copy link
Contributor

@jimmyzhai jimmyzhai commented Aug 25, 2022

Description of PR

Summary:
Ignore the error relating to MACSEC counter reported by syncd of vSonic.

The following log needs ignored in kvmtest-t0-sonic testcase TestFaultHandling.test_link_flap:

Aug 25 11:21:58.390237 vlab-02 ERR syncd#syncd: :- collectData: Failed to get attr of SAI_OBJECT_TYPE_MACSEC_SA 0x5c000000000dd0: -1

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012

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

@jimmyzhai jimmyzhai requested a review from Pterosaur August 25, 2022 12:39
@jimmyzhai jimmyzhai merged commit 83d7643 into sonic-net:master Aug 25, 2022
jimmyzhai added a commit that referenced this pull request Aug 27, 2022
* To follow #6224, update ignore log pattern for MACSEC counter, which is much common.
allen-xf pushed a commit to allen-xf/sonic-mgmt that referenced this pull request Oct 28, 2022
Ignore the error relating to MACSEC counter reported by syncd of vSonic.
allen-xf pushed a commit to allen-xf/sonic-mgmt that referenced this pull request Oct 28, 2022
* To follow sonic-net#6224, update ignore log pattern for MACSEC counter, which is much common.
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.

2 participants