Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[sonic-mgmt-docker-image] Support ptf dataplane packet poll with multiple ptf nn agents connection #21070

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

w1nda
Copy link
Member

@w1nda w1nda commented Dec 6, 2024

Why I did it

When testing sonic with ptf dataplane connecting multiple ptf nn agents, some cases will fail because of packets queue in ptf were not polled thoroughly. This is a bug or missing feature in ptf: p4lang/ptf#207
as a short term quick fix, this PR will patch the ptf-py3 package and unblock our qualification process.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Support poll all devices in ptf dataplane.

How to verify it

Run tests using ptf dataplane on testbed

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

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)

@w1nda w1nda requested a review from lguohan as a code owner December 6, 2024 07:56
@w1nda w1nda requested review from Blueve and r12f and removed request for lguohan December 6, 2024 07:56
@w1nda
Copy link
Member Author

w1nda commented Dec 11, 2024

/azpw ms_conflict

@w1nda
Copy link
Member Author

w1nda commented Dec 11, 2024

/azp ms_conflict

Copy link

Command 'ms_conflict' is not supported by Azure Pipelines.

Supported commands
  • help:
    • Get descriptions, examples and documentation about supported commands
    • Example: help "command_name"
  • list:
    • List all pipelines for this repository using a comment.
    • Example: "list"
  • run:
    • Run all pipelines or specific pipelines for this repository using a comment. Use this command by itself to trigger all related pipelines, or specify specific pipelines to run.
    • Example: "run" or "run pipeline_name, pipeline_name, pipeline_name"
  • where:
    • Report back the Azure DevOps orgs that are related to this repository and org
    • Example: "where"

See additional documentation.

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@w1nda w1nda force-pushed the patch-ptf-for-multi-nn-agents branch from 859c514 to 13def8a Compare December 18, 2024 07:16
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@w1nda w1nda removed request for qiluo-msft and xumia December 18, 2024 07:17
@w1nda w1nda changed the title [ptf-py3] Support ptf dataplane packet poll with multiple ptf nn agents connection [sonic-mgmt-docker-image] Support ptf dataplane packet poll with multiple ptf nn agents connection Dec 23, 2024
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants