Skip to content

Fix test_bgp_suppress_fib.py for t1-f2-d10u8 topo#22116

Merged
bingwang-ms merged 1 commit intosonic-net:masterfrom
markx-arista:fix-test-bgp-suppress-fib-for-t1-f2-topo
Feb 5, 2026
Merged

Fix test_bgp_suppress_fib.py for t1-f2-d10u8 topo#22116
bingwang-ms merged 1 commit intosonic-net:masterfrom
markx-arista:fix-test-bgp-suppress-fib-for-t1-f2-topo

Conversation

@markx-arista
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Update packet verification and port handling to support portchannels with multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

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

  • 202503
  • 202511

Approach

What is the motivation for this PR?

test_bgp_suppress_fib.py failed on t1-f2-d10u8 topo

How did you do it?

Update packet verification and port handling to support portchannels with multiple ethernet interfaces

How did you verify/test it?

test_bgp_suppress_fib.py passed on t1-f2-d10u8 topo

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

@markx-arista markx-arista force-pushed the fix-test-bgp-suppress-fib-for-t1-f2-topo branch from 2dbaab7 to 833f0be Compare January 23, 2026 22:35
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sdszhang sdszhang added Request for msft-202503 Branch Request for 202511 branch Request to backport a change to 202511 branch labels Feb 2, 2026
@sdszhang
Copy link
Copy Markdown
Contributor

sdszhang commented Feb 4, 2026

@markx-arista can you resolve the conflict?

@markx-arista
Copy link
Copy Markdown
Contributor Author

markx-arista commented Feb 4, 2026

@markx-arista can you resolve the conflict?

Azure/sonic-mgmt.msft#1005

@sdszhang
Copy link
Copy Markdown
Contributor

sdszhang commented Feb 4, 2026

@markx-arista can you resolve the conflict?

Azure/sonic-mgmt.msft#1005

Thanks for the 202503 cherry-pick. Can you fix the conflict in this master PR?
image

Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@markx-arista
Copy link
Copy Markdown
Contributor Author

@markx-arista can you resolve the conflict?

Azure/sonic-mgmt.msft#1004

@markx-arista can you resolve the conflict?

Azure/sonic-mgmt.msft#1005

Thanks for the 202503 cherry-pick. Can you fix the conflict in this master PR? image

Thanks, updated

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms bingwang-ms merged commit 5a85172 into sonic-net:master Feb 5, 2026
22 checks passed
bingwang-ms added a commit to Azure/sonic-mgmt.msft that referenced this pull request Feb 5, 2026
<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md

Please provide following information to help code review process a bit
easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should
reviewer start? background context?
- List any dependencies that are required for this change.
-->

Summary:
Update packet verification and port handling to support portchannels
with multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Manually cherry-picked from
sonic-net/sonic-mgmt#22116
Fixes # (issue)

### Type of change

<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

- [ ] Bug fix
- [ ] Testbed and Framework(new/improvement)
- [ ] New Test case
    - [ ] Skipped for non-supported platforms
- [x] Test case improvement


### Back port request
- [x] 202503
- [x] 202511

### Approach
#### What is the motivation for this PR?
test_bgp_suppress_fib.py failed on t1-f2-d10u8 topo

#### How did you do it?
Update packet verification and port handling to support portchannels
with multiple ethernet interfaces

#### How did you verify/test it?
test_bgp_suppress_fib.py passed on t1-f2-d10u8 topo

#### Any platform specific information?

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

### Documentation
<!--
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
Anirudh-nokia pushed a commit to Anirudh-nokia/sonic-mgmt-fork that referenced this pull request Feb 6, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.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
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
Signed-off-by: nnelluri-cisco <nnelluri@cisco.com>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
Signed-off-by: Zhuohui Tan <zhuohui.tan@amd.com>
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 12, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
Signed-off-by: Ravali Yeluri (WIPRO LIMITED) <v-ryeluri@microsoft.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
Signed-off-by: Abhishek <abhishek@nexthop.ai>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 19, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
@ZhaohuiS
Copy link
Copy Markdown
Contributor

@bingwang-ms do we need this backported into 202511?

ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
Update packet verification and port handling to support portchannels with
multiple ethernet interfaces on T0 VMs in t1-f2-d10u8 topology

Signed-off-by: markxiao <markxiao@arista.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.

5 participants