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

Ignore Port Counter laganalyzer error in test_iface_namingmode #16073

Conversation

veronica-arista
Copy link
Contributor

Description of PR

Summary:
Loganalyzer error ERR syncd#syncd: :- collectData: Failed to get stats of Port Counter in test_iface_namingmode.

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, since we observe this error only with test_iface_namingmode.

Fixes # (issue)

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Confirmed this resolved test_iface_namingmode fails on our t2 testbed.

Any platform specific information?

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

Documentation

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.
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Contributor

/Azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rlhui
Copy link

rlhui commented Jan 3, 2025

/Azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Contributor

arlakshm commented Jan 4, 2025

/Azp run Azure.sonic-mgmt

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rlhui
Copy link

rlhui commented Jan 7, 2025

/Azp run Azure.sonic-mgmt

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rlhui
Copy link

rlhui commented Jan 15, 2025

@kenneth-arista , @veronica-arista please help to get the build passed? thanks

@veronica-arista
Copy link
Contributor Author

/Azp run Azure.sonic-mgmt

Copy link

Commenter does not have sufficient privileges for PR 16073 in repo sonic-net/sonic-mgmt

@arlakshm
Copy link
Contributor

/Azp run Azure.sonic-mgmt

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kenneth-arista
Copy link
Contributor

@kenneth-arista , @veronica-arista please help to get the build passed? thanks

The failure appears to be build pipeline specific. For example, see https://elastictest.org/scheduler/publictestplan/67883a3846e46d15f7487fec. The problem is that DUT host unreachable for vms-kvm-dual-t0 on running test_pretest.py|||vms-kvm-dual-t0_391845. What can we do to resolve this?

@rlhui
Copy link

rlhui commented Feb 12, 2025

@arlakshm please help?

@rlhui
Copy link

rlhui commented Feb 27, 2025

/Azp run Azure.sonic-mgmt

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rlhui
Copy link

rlhui commented Feb 27, 2025

@LinJin23 , would you please help on this one?

@LinJin23
Copy link

Hi @veronica-arista , could you sync the latest sonic-net:master branch, merge it into branch veronica-arista:master-fix-loganalyzer-test_iface_namin, and push the changes again? The outdated sonic-net:master branch seems to be affecting the pipeline display.

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rlhui rlhui merged commit 852aefd into sonic-net:master Mar 2, 2025
14 checks passed
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants