Skip to content

Add t0-118 topo test configs#16659

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
dayouliu1:t0-118-test-configs
Feb 25, 2025
Merged

Add t0-118 topo test configs#16659
StormLiangMS merged 1 commit intosonic-net:masterfrom
dayouliu1:t0-118-test-configs

Conversation

@dayouliu1
Copy link
Copy Markdown
Contributor

@dayouliu1 dayouliu1 commented Jan 23, 2025

Description of PR

Summary:
Add Arista-7260CX3-D108C10 t0-118 test configs (emulate t0-116 configs)

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?

Add t0-118 test configs to sonic-mgmt

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?

Arista-7260CX3-D108C10

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@dayouliu1 dayouliu1 force-pushed the t0-118-test-configs branch from a08301c to 8107f5e Compare January 24, 2025 18:19
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@StormLiangMS
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Add Arista-7260CX3-D108C10 t0-118 test configs (emulate t0-116 configs)
@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

@StormLiangMS

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.

Thanks for the update for t0-118. Let's run in on master branch first, if no further regression, we can backport it to 202411 and 202405.

@ZhaohuiS
Copy link
Copy Markdown
Contributor

@StormLiangMS could you please help to review and merge it?

Copy link
Copy Markdown
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 9ac7ad4 into sonic-net:master Feb 25, 2025
17 checks passed
@StormLiangMS
Copy link
Copy Markdown
Collaborator

Hi @dayouliu1 thanks for address this, for the 202411 and 202405 cherrypick, could you run tests with 202405 and 202411 branches and update the results to avoid regression?

nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Mar 4, 2025
What is the motivation for this PR?
Add t0-118 test configs to sonic-mgmt

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?
Arista-7260CX3-D108C10
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Mar 15, 2025
What is the motivation for this PR?
Add t0-118 test configs to sonic-mgmt

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?
Arista-7260CX3-D108C10
@dayouliu1
Copy link
Copy Markdown
Contributor Author

@StormLiangMS We have passing runs on 202411, can we backport this to 202411?

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dayouliu1 PR conflicts with 202405 branch

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dayouliu1 PR conflicts with 202411 branch

@ZhaohuiS
Copy link
Copy Markdown
Contributor

ZhaohuiS commented Apr 9, 2025

@dayouliu1 there are conflicts for 202405 and 202411 branch, could you please submit the separate PRs manually?

StormLiangMS pushed a commit that referenced this pull request Apr 24, 2025
What is the motivation for this PR?
Manually backport t0-118 test configs (#16659) to 202411

How did you do it?
How did you verify/test it?
Ran test suite on t0-118 supported DUT
StormLiangMS pushed a commit that referenced this pull request Apr 24, 2025
What is the motivation for this PR?
Manually backport t0-118 test configs (#16659) to 202405
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.

4 participants