Skip to content

Fix dut_port_config list/dict inconsistency in traffic_generation.py#25490

Open
MahekPatel-2403 wants to merge 1 commit into
sonic-net:masterfrom
MahekPatel-2403:fix-dut-port-config
Open

Fix dut_port_config list/dict inconsistency in traffic_generation.py#25490
MahekPatel-2403 wants to merge 1 commit into
sonic-net:masterfrom
MahekPatel-2403:fix-dut-port-config

Conversation

@MahekPatel-2403

Copy link
Copy Markdown

Description of PR

Fixes the inconsistency in multi_base_traffic_config() where
dut_port_config was initialized as a list but later accessed as a
dictionary using "Tx" and "Rx" keys.

Changes:

  • Update dut_port_config initialization to use a dictionary
  • Update TX/RX peer port insertion logic
  • Update function documentation to reflect the correct structure

Summary:
Fixes #22197

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jun 19, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: MahekPatel-2403 / name: Mahek Patel (5e39ff4)

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

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

Signed-off-by: Mahek Patel <mahekharivansh2008@gmail.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

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

@mssonicbld

Copy link
Copy Markdown
Collaborator

Hi, there are workflow run(s) waiting for approval, you may be first-time contributor. I will notify maintainers to help approve once PR is approved. Thanks!

---Powered by SONiC BuildBot

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.

Bug:snappi: traffic_generation.py: dut_port_config is used as both list and dict.

2 participants