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

Ignoring updating switch mac addr log more broadly #16559

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arista-nwolfe
Copy link
Contributor

We still see this log cause LogAnalyzer to fail in qos tests, so moving this ignore logre to be more broad.

E.G. qos/test_qos_sai.py::TestQosSai::testQosSaiPfcXoffLimit[multi_dut_shortlink_to_shortlink-xoff_1]

Failed: Processes "['analyze_logs--<MultiAsicSonicHost cmp227-4>']" failed with exit code "1"
Exception:
match: 1
expected_match: 0
expected_missing_match: 0

Match Messages:
2024 Dec 16 15:37:17.167501 cmp227-4 ERR syncd0#syncd: message repeated 3 times: [ [06:00.0] SAI_API_SWITCH:brcm_sai_set_switch_attribute:5087 updating switch mac addr failed with error -2.]

Traceback:
Traceback (most recent call last):
  File "/data/tests/common/helpers/parallel.py", line 35, in run
    Process.run(self)
  File "/usr/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/data/tests/common/helpers/parallel.py", line 248, in wrapper
    target(*args, **kwargs)
  File "/data/tests/common/plugins/loganalyzer/__init__.py", line 45, in analyze_logs
    dut_analyzer.analyze(markers[node.hostname], fail_test, store_la_logs=store_la_logs)
  File "/data/tests/common/plugins/loganalyzer/loganalyzer.py", line 409, in analyze
    self._verify_log(analyzer_summary)
  File "/data/tests/common/plugins/loganalyzer/loganalyzer.py", line 140, in _verify_log
    raise LogAnalyzerError(result_str)
tests.common.plugins.loganalyzer.loganalyzer.LogAnalyzerError: match: 1
expected_match: 0
expected_missing_match: 0

Match Messages:
2024 Dec 16 15:37:17.167501 cmp227-4 ERR syncd0#syncd: message repeated 3 times: [ [06:00.0] SAI_API_SWITCH:brcm_sai_set_switch_attribute:5087 updating switch mac addr failed with error -2.]

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

We still see this log cause LogAnalyzer to fail in qos tests, so moving
this ignore logre to be more broad.
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants