Skip to content

Ignore error messages when querying debug counter#12174

Merged
bingwang-ms merged 1 commit intosonic-net:masterfrom
theasianpianist:ignore-debug-log
Apr 1, 2024
Merged

Ignore error messages when querying debug counter#12174
bingwang-ms merged 1 commit intosonic-net:masterfrom
theasianpianist:ignore-debug-log

Conversation

@theasianpianist
Copy link
Copy Markdown
Contributor

@theasianpianist theasianpianist commented Mar 26, 2024

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

Some test cases involving creating/deleting debug counters fail the log analyzer due to the below message:

ERR syncd#syncd: :- collectData: Failed to get stats of Port Debug Counter 0x100000001: -1

This is because there is a short transient period (1-2 sec) after the counter is deleted where the device is still trying to get the counter stats. Once the system becomes stable again the error messages stop.

See ADO 26161007

How did you do it?

Add this message to the loganalyzer common ignore list

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@theasianpianist theasianpianist marked this pull request as draft March 27, 2024 23:07
@theasianpianist theasianpianist marked this pull request as ready for review March 27, 2024 23:47
@theasianpianist theasianpianist requested a review from a team March 27, 2024 23:48
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202311: #12282

mssonicbld pushed a commit that referenced this pull request Apr 3, 2024
veronica-arista added a commit to veronica-arista/sonic-mgmt that referenced this pull request Dec 14, 2024
Similar issue to sonic-net#12174 in that counters
stats continue to be polled in an unstable state.  In the test_iface_namingmode.py
case, TestConfigInterface resets the interface configurations at the end, and these
syslog errors only appear after the test begins the teardown process.

Adding a loganalyzer ignore for this message to this test to fix.
rlhui pushed a commit that referenced this pull request Mar 2, 2025
Similar issue to #12174 in that counters
stats continue to be polled in an unstable state.  In the test_iface_namingmode.py
case, TestConfigInterface resets the interface configurations at the end, and these
syslog errors only appear after the test begins the teardown process.

Adding a loganalyzer ignore for this message to this test to fix.
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Mar 4, 2025
Similar issue to sonic-net#12174 in that counters
stats continue to be polled in an unstable state.  In the test_iface_namingmode.py
case, TestConfigInterface resets the interface configurations at the end, and these
syslog errors only appear after the test begins the teardown process.

Adding a loganalyzer ignore for this message to this test to fix.
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Mar 15, 2025
Similar issue to sonic-net#12174 in that counters
stats continue to be polled in an unstable state.  In the test_iface_namingmode.py
case, TestConfigInterface resets the interface configurations at the end, and these
syslog errors only appear after the test begins the teardown process.

Adding a loganalyzer ignore for this message to this test to fix.
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.

6 participants