Fix/nonlinear dataplane downtime#21936
Merged
bingwang-ms merged 6 commits intosonic-net:masterfrom Jan 20, 2026
Merged
Conversation
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
10 tasks
r12f
reviewed
Jan 15, 2026
Collaborator
|
straightforward change. will sign off, once comments are addressed. |
Collaborator
|
Picking to 202412: Azure/sonic-mgmt.msft#982 |
Collaborator
|
@PriyanshTratiya PR conflicts with 202511 branch |
10 tasks
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com> Signed-off-by: Yael Tzur <ytzur@nvidia.com>
abhishek-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Feb 6, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
Anirudh-nokia
pushed a commit
to Anirudh-nokia/sonic-mgmt-fork
that referenced
this pull request
Feb 6, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com> Signed-off-by: ayya <anirudh.ayya@nokia.com>
Contributor
|
Could you help cherry-pick the fix to 202511 branch? Thanks |
PriyanshTratiya
added a commit
to PriyanshTratiya/sonic-mgmt
that referenced
this pull request
Feb 9, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
12 tasks
nnelluri-cisco
pushed a commit
to nnelluri-cisco/sonic-mgmt
that referenced
this pull request
Feb 12, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com> Signed-off-by: nnelluri-cisco <nnelluri@cisco.com>
rraghav-cisco
pushed a commit
to rraghav-cisco/sonic-mgmt
that referenced
this pull request
Feb 13, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com> Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
PriyanshTratiya
added a commit
to PriyanshTratiya/sonic-mgmt
that referenced
this pull request
Feb 14, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
Contributor
Author
|
vmittal-msft
pushed a commit
that referenced
this pull request
Feb 18, 2026
anilal-amd
pushed a commit
to anilal-amd/anilal-forked-sonic-mgmt
that referenced
this pull request
Feb 19, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com> Signed-off-by: Zhuohui Tan <zhuohui.tan@amd.com>
abhishek-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Mar 17, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com> Signed-off-by: Abhishek <abhishek@nexthop.ai>
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Mar 27, 2026
* ptf dataplane cleaners for in between test runs Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
Summary:
Fixes # (issue)
This PR addresses non‑linear dataplane downtime behavior observed in high‑scale BGP IPv6 scenarios when running the port and session flapping tests. When the number of connections to flap doubled, the dataplane downtime increased by 450x.
This change refines the tests and helper logic to ensure that downtime measurements:
Type of change
Back port request
Approach
What is the motivation for this PR?
While validating high‑scale BGP convergence, flap, and route‑programming tests, we observed that:
These issues were traced to how the tests were executed sequentially while the PTF dataplane packet‑filtering/counter state was never cleared between runs. As a result, masks and counters kept accumulating over time, so that each subsequent run especially those with a larger number of ports to flap saw an artificially inflated dataplane downtime.
In other words, the measured non‑linear increase in downtime was caused by PTF dataplane state rather than actual BGP control‑plane behavior. The goal of this PR is to:
How did you do it?
How did you verify/test it?
t0-isolated-d2u510s2Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation