Skip to content

[smartswitch] fix the errors from command execution on DPU#23323

Merged
prabhataravind merged 1 commit intosonic-net:masterfrom
aronovic:fix_dash_pl_redirect_and_vnet
Mar 26, 2026
Merged

[smartswitch] fix the errors from command execution on DPU#23323
prabhataravind merged 1 commit intosonic-net:masterfrom
aronovic:fix_dash_pl_redirect_and_vnet

Conversation

@aronovic
Copy link
Contributor

@aronovic aronovic commented Mar 25, 2026

Description of PR

Summary:
Fixes #23311
Fixes this issue:
vnet and PL redirect DASH Tests are failing because this command failure: config loopback add Loopback0', 'config int ip add Loopback0 10.1.0.5/32', 'ip route replace default via '

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?

Need a fix for vnet and PL redirect DASH tests that are failing because this command failure: config loopback add Loopback0', 'config int ip add Loopback0 10.1.0.5/32', 'ip route replace default via '
This is because after topology changes introduced in PR 22902, npu_data_port_ip is not set.

How did you do it?

Skipped the command execution when those fields are empty

How did you verify/test it?

Run on MtFuji topology

Any platform specific information?

MtFuji

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

cmono_t1

Documentation

N/A

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@aronovic aronovic force-pushed the fix_dash_pl_redirect_and_vnet branch from dc25301 to ea2ed16 Compare March 25, 2026 21:52
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@yxieca yxieca left a comment

Choose a reason for hiding this comment

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

AI agent on behalf of Ying. Reviewed; no issues found.

@prabhataravind prabhataravind merged commit 0ac2220 into sonic-net:master Mar 26, 2026
15 checks passed
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

4 participants