Skip to content

[action] [PR:21617] Enhance PTF function at scale testbed#22067

Open
mssonicbld wants to merge 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/21617
Open

[action] [PR:21617] Enhance PTF function at scale testbed#22067
mssonicbld wants to merge 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/21617

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

On Scale testbed there are 2 main differences:

  1. There are much more PTF ports
  2. There are much more OVS and interface configurations

Those lead to:

  1. NIC would take more time to handle the receiving packets
  2. PTF could take more time to process

As a result:

  1. More time to spend in ptf function testutils.verify_packet

Summary:
Fixes # (issue)

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?

To increase the PTF traffic handling stability in the scale testbed

How did you do it?

Add more wait time in ptf function testutils.verify_packet

How did you verify/test it?

Run it in the local scale testbed

Any platform specific information?

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

Documentation

On Scale testbed there are 2 main differences:
1. There are much more PTF ports
2. There are much more OVS and interface configurations

Those lead to:
1. NIC would take more time to handle the receiving packets
2. PTF could take more time to process

As a result:
1. More time to spend in ptf function testutils.verify_packet

Change-Id: I8cb9b0bf983f7a759b52f0770750788a5e4fe362

Signed-off-by: echuawu <chuanw@nvidia.com>
@mssonicbld
Copy link
Collaborator Author

Original PR: #21617

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants