Skip to content

Move some srv6 functions to srv6_helper.py#18892

Merged
bingwang-ms merged 1 commit intosonic-net:masterfrom
weiguo-nvidia:move_srv6_function_0610
Jun 30, 2025
Merged

Move some srv6 functions to srv6_helper.py#18892
bingwang-ms merged 1 commit intosonic-net:masterfrom
weiguo-nvidia:move_srv6_function_0610

Conversation

@weiguo-nvidia
Copy link
Copy Markdown
Contributor

Description of PR

Summary: Move some srv6 functions to srv6_helper.py
Fixes #
Move some srv6 functions from tests/srv6/srv6_utils.py to tests/common/helpers/srv6_helper.py so that other functions can call srv6 related functions when doing integration tests

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?

How did you do it?

How did you verify/test it?

Any platform specific information?

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).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

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

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

Hi @bingwang-ms @yxieca , could you help merge this PR, thanks!

@bingwang-ms bingwang-ms merged commit c2a3553 into sonic-net:master Jun 30, 2025
21 checks passed
@bingwang-ms
Copy link
Copy Markdown
Collaborator

@r12f Please handle cherry-pick.

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19292

@r12f
Copy link
Copy Markdown
Collaborator

r12f commented Jul 2, 2025

hi @weiguo-nvidia , do you mind to help manual pick this PR to 202412?

yejianquan pushed a commit that referenced this pull request Jul 2, 2025
Description of PR
Summary: Move some srv6 functions to srv6_helper.py
Fixes #
Move some srv6 functions from tests/srv6/srv6_utils.py to tests/common/helpers/srv6_helper.py so that other functions can call srv6 related functions when doing integration tests

Co-authored-by: weguo-NV <[email protected]>
@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

Hi @r12f
I have submitted PR Azure/sonic-mgmt.msft#487 to 202412 branch, could you help review? Thanks!

r12f pushed a commit to Azure/sonic-mgmt.msft that referenced this pull request Jul 3, 2025
#### Description of PR ####
Summary: Move some srv6 functions to srv6_helper.py
Fixes #
Move some srv6 functions from tests/srv6/srv6_utils.py to
tests/common/helpers/srv6_helper.py so that other functions can call
srv6 related functions when doing integration tests

Origin PR: sonic-net/sonic-mgmt#18892
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
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.

7 participants