Skip to content

Backport t0-118 test configs to 202405#18015

Merged
StormLiangMS merged 2 commits intosonic-net:202405from
dayouliu1:202405
Apr 24, 2025
Merged

Backport t0-118 test configs to 202405#18015
StormLiangMS merged 2 commits intosonic-net:202405from
dayouliu1:202405

Conversation

@dayouliu1
Copy link
Copy Markdown
Contributor

@dayouliu1 dayouliu1 commented Apr 16, 2025

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

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

Manually backport t0-118 test configs (#16659) to 202405

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

Backport t0-118 test config changes to 202405
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

::/0 [24 25] [26 27] [28 29] [30 31]
{% elif testbed_type == 't0-118' %}
0.0.0.0/0 [0, 1] [22 23] [24 25] [26 27] [28 29]
::/0 [0, 1] [22 23] [24 25] [26 27] [28 29]
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.

@dayouliu1 according to the topo-116 and topo-118 yml file,
for 10-118, it should be 0.0.0.0/0 [22 23] [24 25] [26 27] [28 29], [0, 1] should not be included.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch!

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.

@dayouliu1 ipv6 also needs to update

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@ZhaohuiS updated!

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@dayouliu1 dayouliu1 requested a review from ZhaohuiS April 17, 2025 16:18
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@dayouliu1
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

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

@dayouliu1
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
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.

Looks good to me now

@StormLiangMS StormLiangMS merged commit 9aeb337 into sonic-net:202405 Apr 24, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants