Skip to content

ci: enable t1-lag-vpp as optional test#21944

Merged
wangxin merged 1 commit intosonic-net:masterfrom
auspham:austinpham/36369621-enable-vpp-test-on-pr
Jan 20, 2026
Merged

ci: enable t1-lag-vpp as optional test#21944
wangxin merged 1 commit intosonic-net:masterfrom
auspham:austinpham/36369621-enable-vpp-test-on-pr

Conversation

@auspham
Copy link
Copy Markdown
Contributor

@auspham auspham commented Jan 15, 2026

Description of PR

Summary: This enable VPP data plane testing. For now, we keep it optional
Fixes # (issue) 3636962

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
  • 202511

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

@auspham auspham requested a review from wangxin as a code owner January 15, 2026 04:01
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

liushilongbuaa
liushilongbuaa previously approved these changes Jan 16, 2026
Copy link
Copy Markdown
Contributor

@liushilongbuaa liushilongbuaa left a comment

Choose a reason for hiding this comment

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

LGTM

KVM_IMAGE_BUILD_PIPELINE_ID: "2818"
COMMON_EXTRA_PARAMS: "--disable_sai_validation --disable_loganalyzer"
STOP_ON_FAILURE: "False"
EXPECTED_RESULT: "SUCCESS"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I remember in the previous PR, we need to revert the expected_result to avoid pipeline issue, can you double confirm?

@yejianquan
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@xwjiang-ms
Copy link
Copy Markdown
Contributor

Suggest to set default expected result to success here:


Then set expected result in GLOBAL_PARAMS to empty here: sonic-mgmt/.azure-pipelines/baseline_test/baseline.test.mgmt.public.yml at master · sonic-net/sonic…

Signed-off-by: Austin Pham <austinpham@microsoft.com>
@auspham auspham force-pushed the austinpham/36369621-enable-vpp-test-on-pr branch from 35d36dc to 4cd593e Compare January 19, 2026 08:20
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@yejianquan yejianquan left a comment

Choose a reason for hiding this comment

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

LGTM

@wangxin wangxin merged commit 9717f84 into sonic-net:master Jan 20, 2026
22 checks passed
PriyanshTratiya pushed a commit to PriyanshTratiya/sonic-mgmt that referenced this pull request Jan 21, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
saravanan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 22, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: Saravanan Sellappa <saravanan@nexthop.ai>
justin-oliver pushed a commit to justin-oliver/sonic-mgmt that referenced this pull request Jan 26, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: Yael Tzur <ytzur@nvidia.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Anirudh-nokia pushed a commit to Anirudh-nokia/sonic-mgmt-fork that referenced this pull request Feb 6, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: ayya <anirudh.ayya@nokia.com>
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Feb 12, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: nnelluri-cisco <nnelluri@cisco.com>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: Zhuohui Tan <zhuohui.tan@amd.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
<!--
 Please make sure you've read and understood our contributing guidelines:
 https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

 failure_prs.log Make sure all your commits include a signature generated with `git commit -s` **

 If this is a bug fix, make sure your description includes "fixes #xxxx", or
 "closes #xxxx" or "resolves #xxxx"

 Please provide the following information:
-->

#### Why I did it

Enhance yang model for networking-metadata.

Adding a new field in device metadata YANG model to distinguish the management type of the device. Depends on the users, this field can be used to tell the purpose of deployment or who/how the device is being managed.

##### Work item tracking
- Microsoft ADO **(number only)**:

#### How I did it

This change adds a new field called mgmt_type for this purpose.

#### How to verify it

Unit test.

#### Which release branch to backport (provide reason below if selected)

<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- e.g.
- [x] 202006
-->

- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
- [ ] 202205
- [ ] 202211
- [ ] 202305
- [X] 202405
- [X] 202411

#### Tested branch (Please provide the tested image version)

<!--
- Please provide tested image version
- e.g.
- [x] 20201231.100
-->

- [ ] <!-- image version 1 -->
- [ ] <!-- image version 2 -->

#### Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->

<!--
 Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.
-->

#### Link to config_db schema for YANG module changes
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
-->

#### A picture of a cute animal (not mandatory but encouraged)
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
Signed-off-by: Abhishek <abhishek@nexthop.ai>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
This enable VPP data plane testing. For now, we keep it optional

Signed-off-by: Austin Pham <austinpham@microsoft.com>
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.

6 participants