Skip to content

Add multi-asic support to test_pktgen.py#19399

Merged
abdosi merged 1 commit intosonic-net:masterfrom
arista-nwolfe:master-test_pktgen-multi-asic
Jul 16, 2025
Merged

Add multi-asic support to test_pktgen.py#19399
abdosi merged 1 commit intosonic-net:masterfrom
arista-nwolfe:master-test_pktgen-multi-asic

Conversation

@arista-nwolfe
Copy link
Contributor

On multi-asic systems the kernel interfaces of the front panel ports live in the per-asic namespace.

E.G.

> ifconfig | grep -i Ethernet0

> sudo ip netns exec asic0 ifconfig | grep Ethernet0
Ethernet0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 9100

Meaning commands like echo 'add_device Ethernet0' > /proc/net/pktgen/kpktgend_0 need to be run within the asic namespace.

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
  • msft-202503
  • 202505

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe
Copy link
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abdosi
Copy link
Contributor

abdosi commented Jul 16, 2025

@arista-nwolfe : Please check if we can back port to 202405 and see if it passes.

@abdosi abdosi merged commit 739e859 into sonic-net:master Jul 16, 2025
20 checks passed
@arista-nwolfe
Copy link
Contributor Author

@arista-nwolfe : Please check if we can back port to 202405 and see if it passes.

@abdosi I confirmed this change does pass on multi-asic on 202405.
However, we first need to cast #14460 to 202405.
Also we'll need #18353 in 202405 to remove the skip conditional.

nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
yejianquan added a commit to Azure/sonic-mgmt.msft that referenced this pull request Oct 14, 2025
This is a cherry-pick PR to include PR:
sonic-net/sonic-mgmt#19399

This PR has a dependency PRs, which are included in this PR as well:

sonic-net/sonic-mgmt#14460
sonic-net/sonic-mgmt#18353

signed-0ff-by: jianquanye@microsoft.com
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
Signed-off-by: Yael Tzur <ytzur@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants