Skip to content

[DASH] Use VLAN intf for DPU dataplane intf when available#23352

Open
theasianpianist wants to merge 1 commit intosonic-net:masterfrom
theasianpianist:ss-vlan-ip
Open

[DASH] Use VLAN intf for DPU dataplane intf when available#23352
theasianpianist wants to merge 1 commit intosonic-net:masterfrom
theasianpianist:ss-vlan-ip

Conversation

@theasianpianist
Copy link
Copy Markdown
Contributor

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

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

Approach

What is the motivation for this PR?

#22902 changed the default smartswitch config to put all DPU dataplane interfaces on the NPU in a VLAN. When trying to reach the DPU dataplane, the appropriate interface is now the VLAN interface.

How did you do it?

Once the individual DPU-facing port is found on the NPU, check if it is a VLAN member. If so, us the VLAN interface instead.

How did you verify/test it?

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

Copy link
Copy Markdown
Contributor

@prabhataravind prabhataravind left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@prsunny prsunny left a comment

Choose a reason for hiding this comment

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

lgtm.

Copy link
Copy Markdown
Contributor

@zjswhhh zjswhhh left a comment

Choose a reason for hiding this comment

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

lgtm

@prsunny prsunny enabled auto-merge (squash) March 26, 2026 19:00
@prabhataravind
Copy link
Copy Markdown
Contributor

/azp run Azure.sonic-mgmt

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.

5 participants