Skip to content

[ptf] Improve runtime of verify_no_packet_any for large port sets#25347

Merged
kperumalbfn merged 2 commits intosonic-net:masterfrom
AntonHryshchuk:anton-add-ptf-patch
Feb 12, 2026
Merged

[ptf] Improve runtime of verify_no_packet_any for large port sets#25347
kperumalbfn merged 2 commits intosonic-net:masterfrom
AntonHryshchuk:anton-add-ptf-patch

Conversation

@AntonHryshchuk
Copy link
Copy Markdown
Contributor

Why I did it

To reduce the runtime of tests

How I did it

Created PR for PTF repository, and added a patch to sonic-buildimage
p4lang/ptf#231

How to verify it

Run drop_packets tests in sonic-mgmt.

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@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).

@vivekrnv
Copy link
Copy Markdown
Contributor

vivekrnv commented Feb 4, 2026

@saiarcot895 Please help review

Signed-off-by: AntonHryshchuk <antonh@nvidia.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).

@AntonHryshchuk
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@AntonHryshchuk
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@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).

@roy-sror
Copy link
Copy Markdown
Collaborator

roy-sror commented Feb 9, 2026

@lguohan - can you please help in reviewing this one?

@kperumalbfn
Copy link
Copy Markdown
Contributor

@AntonHryshchuk please let us know once the p4lang PR gets merged to the repo, so that we will have same patch.

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Feb 12, 2026

hi @kperumalbfn , we need this PR badly for 202511 full topo qual. the P4 change might take very long time to merge and release, and we can update the patch later once that is merged with any additional changes.

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Feb 12, 2026

this change only affects testing and no real feature impacts.

Copy link
Copy Markdown
Contributor

@kperumalbfn kperumalbfn left a comment

Choose a reason for hiding this comment

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

LGTM, please update the patch if any additional changes made in p4lang PR

@kperumalbfn kperumalbfn merged commit 926801a into sonic-net:master Feb 12, 2026
26 checks passed
@kperumalbfn
Copy link
Copy Markdown
Contributor

@vmittal-msft pls merge to 202511

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202511: #25583

FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Mar 6, 2026
…nic-net#25347)

[ptf] Improve runtimr of verify_no_packet_any for large port sets

Signed-off-by: Feng Pan <fenpan@microsoft.com>
dprital pushed a commit that referenced this pull request Mar 19, 2026
…5347)

[ptf] Improve runtimr of verify_no_packet_any for large port sets

Signed-off-by: dprital <drorp@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants