Skip to content

[validation][202605] VPP PortChannel punt for DHCP relay - #29067

Draft
Xichen96 wants to merge 5 commits into
sonic-net:202605from
Xichen96:dev/xichenlin/vpp-portchannel-punt-202605-validation
Draft

Xichen96 wants to merge 5 commits into
sonic-net:202605from
Xichen96:dev/xichenlin/vpp-portchannel-punt-202605-validation

Conversation

@Xichen96

Copy link
Copy Markdown
Contributor

Why I did it

Validate the merged VPP PortChannel punt fix from
sonic-net/sonic-platform-vpp#257 on the 202605 build before requesting a
release backport.

The current 202605 VPP image pins platform/vpp before that fix. In the
t0-vpp lane, DHCP relay receives no relayed packets on the PortChannel
uplinks (0 != 4). Master images containing #257 pass the same test.

This PR is validation-only. It carries no release/backport labels.

Work item tracking
  • Microsoft ADO (number only): 39310228

How I did it

Update only the platform/vpp submodule pointer:

  • from f6490367171147cf9f314d3c38c9a9410b093c50
  • to 53a0712b199b5cba54b6a433ca0b8c688d2c60cc

The target commit is the merged result of
sonic-net/sonic-platform-vpp#257. Its punt-via-member handling redirects
control-plane packets received on a PortChannel to the original member tap,
matching the interfaces where dhcrelay listens.

How to verify it

Run the 202605 buildimage PR pipeline and require:

  • BuildVS vpp to pass;
  • Test impacted-area-kvmtest-t0-vpp by Elastictest to pass;
  • dhcp_relay/test_dhcp_relay.py::test_dhcp_relay_default[isc-relay-agent]
    to receive all four expected relayed DHCPDISCOVER packets.

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

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

N/A — this is a validation-only PR already targeting 202605. Backport
approval will be requested on the original PR only after validation passes.

Tested branch (Please provide the tested image version)

  • Pending PR CI

Description for the changelog

Validate the VPP PortChannel punt fix on the 202605 image.

Link to config_db schema for YANG module changes

N/A.

A picture of a cute animal (not mandatory but encouraged)

N/A.

Signed-off-by: Xichen96 <lukelin0907@gmail.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Advance to the VPP 2606-0.4 package bump that contains sonic-platform-vpp sonic-net#257.

Copilot-Session: f34adeaf-e3b4-4c07-9391-6c62427d8a7c
Signed-off-by: Xichen96 <lukelin0907@gmail.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Advance to sonic-platform-vpp sonic-net#267, which preserves device-input features on LAG parents and publishes VPP 2606-0.5.

Copilot-Session: f34adeaf-e3b4-4c07-9391-6c62427d8a7c
Signed-off-by: Xichen96 <lukelin0907@gmail.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Advance to sonic-platform-vpp sonic-net#272, the final relevant commit in the passing master VPP pointer.

Copilot-Session: f34adeaf-e3b4-4c07-9391-6c62427d8a7c
Signed-off-by: Xichen96 <lukelin0907@gmail.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Combine sonic-sairedis sonic-net#1981 with the platform-vpp sonic-net#257/sonic-net#271 packaged punt-via-member implementation.

Copilot-Session: f34adeaf-e3b4-4c07-9391-6c62427d8a7c
Signed-off-by: Xichen96 <lukelin0907@gmail.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

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.

2 participants