Skip to content

[Arista] Ignore SRV6 ERR log on Arista platforms - #20049

Merged
Blueve merged 2 commits into
sonic-net:masterfrom
lizhijianrd:ingore-arista-err-log
Aug 4, 2025
Merged

[Arista] Ignore SRV6 ERR log on Arista platforms#20049
Blueve merged 2 commits into
sonic-net:masterfrom
lizhijianrd:ingore-arista-err-log

Conversation

@lizhijianrd

@lizhijianrd lizhijianrd commented Aug 4, 2025

Copy link
Copy Markdown
Contributor

Description of PR

Summary:

We got below loganalyzer failure on latest Arista images. Since this ERR log is only for SRV6, I ignore them in loganalyzer:

E               Failed: Got matched syslog in processes "analyze_logs--<MultiAsicSonicHost 720dt-3>" exit code:"1"
E               match: 2
E               expected_match: 0
E               expected_missing_match: 0
E               
E               Match Messages:
E               2025 Aug  2 17:59:33.069264 720dt-3 ERR syncd#syncd: [none] SAI_API_SRV6:_brcm_sai_srv6_config_get:1309 SRV6 unsupported or not enabled on this device
E               
E               2025 Aug  2 17:59:33.069264 720dt-3 ERR syncd#syncd: [none] SAI_API_SWITCH:sai_object_type_get_availability:1002 availablity my_sid 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

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Ignore SRV6 ERR in loganalyzer.

How did you do it?

Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?

Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:

lldp/test_lldp_syncd.py::test_lldp_entry_table_content[720dt-1] PASSED                                                                                                                                                   [ 20%]
lldp/test_lldp_syncd.py::test_lldp_entry_table_after_syncd_orchagent[720dt-1] PASSED                                                                                                                                     [ 40%]
lldp/test_lldp_syncd.py::test_lldp_entry_table_after_flap[720dt-1] PASSED                                                                                                                                                [ 60%]
lldp/test_lldp_syncd.py::test_lldp_entry_table_after_lldp_restart[720dt-1] PASSED                                                                                                                                        [ 80%]
lldp/test_lldp_syncd.py::test_lldp_entry_table_after_reboot[720dt-1] PASSED                                                                                                                                              [100%]
=============================================================================================== 5 passed, 1 warning in 1014.52s (0:16:54) ===============================================================================================

Any platform specific information?

Arista.

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

Documentation

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Comment thread ansible/roles/test/files/tools/loganalyzer/loganalyzer_common_ignore.txt Outdated
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@Gfrom2016 Gfrom2016 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@Blueve
Blueve merged commit 198cc2d into sonic-net:master Aug 4, 2025
20 of 21 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Aug 4, 2025
What is the motivation for this PR?
Ignore SRV6 ERR in loganalyzer.

How did you do it?
Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?
Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:
@mssonicbld

Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #20052

@lizhijianrd
lizhijianrd deleted the ingore-arista-err-log branch August 4, 2025 14:35
mssonicbld pushed a commit that referenced this pull request Aug 4, 2025
What is the motivation for this PR?
Ignore SRV6 ERR in loganalyzer.

How did you do it?
Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?
Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:
Pterosaur added a commit to Azure/sonic-mgmt.msft that referenced this pull request Sep 9, 2025
Temporary solution for tests failing from recent regression as a result
of upgrading to a newer SAI.
Original PR: sonic-net/sonic-mgmt#20049
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
What is the motivation for this PR?
Ignore SRV6 ERR in loganalyzer.

How did you do it?
Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?
Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
What is the motivation for this PR?
Ignore SRV6 ERR in loganalyzer.

How did you do it?
Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?
Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
What is the motivation for this PR?
Ignore SRV6 ERR in loganalyzer.

How did you do it?
Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?
Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
What is the motivation for this PR?
Ignore SRV6 ERR in loganalyzer.

How did you do it?
Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?
Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
What is the motivation for this PR?
Ignore SRV6 ERR in loganalyzer.

How did you do it?
Ignore SRV6 ERR in loganalyzer.

How did you verify/test it?
Verified by testcase test_lldp_syncd.py on Arista-720DT Mx:

Signed-off-by: Yael Tzur <ytzur@nvidia.com>
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.

5 participants