Skip to content

[master][sonic-mgmt]Skip ipv6 cases under everflow/test_everflow_per_interface.py on on Arista-7260CX3 and Arista-7060CX#19500

Merged
StormLiangMS merged 2 commits intosonic-net:masterfrom
pragnya-arista:master-skip-v6-everflow-per-interface
Jul 25, 2025
Merged

[master][sonic-mgmt]Skip ipv6 cases under everflow/test_everflow_per_interface.py on on Arista-7260CX3 and Arista-7060CX#19500
StormLiangMS merged 2 commits intosonic-net:masterfrom
pragnya-arista:master-skip-v6-everflow-per-interface

Conversation

@pragnya-arista
Copy link
Contributor

Description of PR

Summary:
Fixes # (issue)

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?

  • Added further cases to skip as requested in 19308
  • The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096

How did you do it?

How did you verify/test it?

Any platform specific information?

Yes - Arista-7260CX3(TH2)

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

Documentation

everflow/test_everflow_per_interface.py on on Arista-7260CX3
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mramezani95
Copy link
Contributor

@pragnya-arista Can you please skip these tests on "x86_64-arista_7060_cx32s" as well? This platform also has the same SAI issue.

@pragnya-arista
Copy link
Contributor Author

@pragnya-arista Can you please skip these tests on "x86_64-arista_7060_cx32s" as well? This platform also has the same SAI issue.

Okay, sure let me do that

@pragnya-arista
Copy link
Contributor Author

@pragnya-arista Can you please skip these tests on "x86_64-arista_7060_cx32s" as well? This platform also has the same SAI issue.

Okay, sure, let me do that

To not mix up things, I'll open a single, separate review to skip pertinent test cases under test_everflow_per_interface, test_everflow_testbed, and test_everflow_ipv6 cases for x86_64-arista_7060_cx32s @mramezani95

@mramezani95
Copy link
Contributor

@pragnya-arista Can you please skip these tests on "x86_64-arista_7060_cx32s" as well? This platform also has the same SAI issue.

Okay, sure, let me do that

To not mix up things, I'll open a single, separate review to skip pertinent test cases under test_everflow_per_interface, test_everflow_testbed, and test_everflow_ipv6 cases for x86_64-arista_7060_cx32s @mramezani95

@pragnya-arista I have already submitted a PR to skip tests in other files. Here is the PR: #19529

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@ZhaohuiS ZhaohuiS left a comment

Choose a reason for hiding this comment

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

LGTM

@ZhaohuiS
Copy link
Contributor

@pragnya-arista I think Arista-7060CX-32S-D48C8 also hits the same issue, should we also skip for Arista-7060CX-32S-D48C8?

@pragnya-arista
Copy link
Contributor Author

@pragnya-arista I think Arista-7060CX-32S-D48C8 also hits the same issue, should we also skip for Arista-7060CX-32S-D48C8?

19529 should be doing this

@ZhaohuiS
Copy link
Contributor

@pragnya-arista I think Arista-7060CX-32S-D48C8 also hits the same issue, should we also skip for Arista-7060CX-32S-D48C8?

19529 should be doing this

@pragnya-arista #19529 is for test_everflow_ipv6.py test cases not for test_everflow_per_interface.py. I mean test_everflow_per_interface.py also failed on 7060. Should we skip it for 7060 as well in this PR?

@pragnya-arista
Copy link
Contributor Author

@pragnya-arista I think Arista-7060CX-32S-D48C8 also hits the same issue, should we also skip for Arista-7060CX-32S-D48C8?

19529 should be doing this

@pragnya-arista #19529 is for test_everflow_ipv6.py test cases not for test_everflow_per_interface.py. I mean test_everflow_per_interface.py also failed on 7060. Should we skip it for 7060 as well in this PR?

You're right - I'll amend 19500 to skip for both 7260CX3 and 7060
Thanks

@ZhaohuiS
Copy link
Contributor

@pragnya-arista will you update this PR soon? Or do you need a new PR to address 7060? We would like to skip test_everflow_per_interface ASAP for 202505.

@pragnya-arista
Copy link
Contributor Author

pragnya-arista commented Jul 24, 2025

Thanks @ZhaohuiS

To summarise, there are three areas we need to skip ipv6 everflow cases

  • test_everflow_testbed

  • test_everflow_ipv6

  • test_everflow_per_interface

  • 19308 skips ipv6 cases under test_everflow_testbed.py and test_everflow_ipv6.py for 7260CX3 and has merged

  • 19500 skips ipv6 cases under test_everflow_per_interface.py for 7260CX3 and is open

  • 19529 skips ipv6 cases under test_everflow_ipv6.py for 7060CX as well and has merged

  • I will amend 19500 to skip test_everflow_per_interface.py for 7060CX3 as well. I will do this in an hour, and we can merge the changes post-inclusion of skipping 7060CX

  • I will open a new PR for skipping relevant ipv6 cases test_everflow_testbed.py for 7060CX

test_everflow_per_interface.py on Arista-7060CX
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@pragnya-arista
Copy link
Contributor Author

  • I will open a new PR for skipping test_everflow_testbed.py for 7060CX - @ZhaohuiS let me know if I can go ahead with this

I've updated 19500 to skip 7060CX for relevant tests under test_everflow_per_interface.py - kindly review

Thanks

@ZhaohuiS
Copy link
Contributor

  • I will open a new PR for skipping test_everflow_testbed.py for 7060CX - @ZhaohuiS let me know if I can go ahead with this

I've updated 19500 to skip 7060CX for relevant tests under test_everflow_per_interface.py - kindly review

Thanks
Approved.

@pragnya-arista pragnya-arista changed the title [master][sonic-mgmt]Skip ipv6 cases under everflow/test_everflow_per_interface.py on on Arista-7260CX3 [master][sonic-mgmt]Skip ipv6 cases under everflow/test_everflow_per_interface.py on on Arista-7260CX3 and Arista-7060CX Jul 24, 2025
@pragnya-arista
Copy link
Contributor Author

Thanks @ZhaohuiS

To summarise, there are three areas we need to skip ipv6 everflow cases

  • test_everflow_testbed
  • test_everflow_ipv6
  • test_everflow_per_interface
  • 19308 skips ipv6 cases under test_everflow_testbed.py and test_everflow_ipv6.py for 7260CX3 and has merged
  • 19500 skips ipv6 cases under test_everflow_per_interface.py for 7260CX3 and is open
  • 19529 skips ipv6 cases under test_everflow_ipv6.py for 7060CX as well and has merged
  • I will amend 19500 to skip test_everflow_per_interface.py for 7060CX3 as well. I will do this in an hour, and we can merge the changes post-inclusion of skipping 7060CX
  • I will open a new PR for skipping relevant ipv6 cases test_everflow_testbed.py for 7060CX
    I will open a new PR for skipping relevant ipv6 cases test_everflow_testbed.py for 7060CX ---?
    Opened 19811 for this

Lise of all PRs skipping all pertinent IPv6 everflow cases for 7260CX3 and 7060CX

  • 19308 skips ipv6 cases under test_everflow_testbed.py and test_everflow_ipv6.py for 7260CX3
  • 19500 skips ipv6 cases under test_everflow_per_interface.py for 7260CX3 and and 7060CX
  • 19529 skips ipv6 cases under test_everflow_ipv6.py for 7060CX
  • 19811 skips ipv6 cases under test_everflow_testbed.py for 7060CX

Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit 0d38d8b into sonic-net:master Jul 25, 2025
18 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 25, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #19837

mssonicbld pushed a commit that referenced this pull request Jul 25, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)

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
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
…interface.py on on Arista-7260CX3 and Arista-7060CX (sonic-net#19500)

What is the motivation for this PR?
Added further cases to skip as requested in 19308
The original issue that requires these cases to be skipped is being tracked in 627 and public issue 19096
How did you do it?
How did you verify/test it?
Any platform specific information?
Yes - Arista-7260CX3(TH2)

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.

6 participants