Skip to content

Disable ipv6 traffic from ptf to dut for two qos cases because it will introduce noise#20416

Merged
roy-sror merged 2 commits intosonic-net:masterfrom
JibinBao:qos_disable_ipv6_traffic
Oct 28, 2025
Merged

Disable ipv6 traffic from ptf to dut for two qos cases because it will introduce noise#20416
roy-sror merged 2 commits intosonic-net:masterfrom
JibinBao:qos_disable_ipv6_traffic

Conversation

@JibinBao
Copy link
Copy Markdown
Contributor

Description of PR

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

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

Approach

What is the motivation for this PR?

Stabilize testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy]

How did you do it?

Disable the ipv6 traffic from ptf to dut for the two cases.

How did you verify/test it?

Run the two case many times

Any platform specific information?

Any

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao JibinBao force-pushed the qos_disable_ipv6_traffic branch from 0ebb0ca to c57abc2 Compare August 27, 2025 03:12
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao JibinBao changed the title Disable ipv6 traffic from ptf to dut for two qos cases because it will cause noise Disable ipv6 traffic from ptf to dut for two qos cases because it will introduce noise Aug 27, 2025
@JibinBao JibinBao force-pushed the qos_disable_ipv6_traffic branch from c57abc2 to ea5ea4d Compare August 27, 2025 03:17
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

When IPv6 is not enabled on the PTF, it sends RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of  the egress port buffer is occupied. Since the PTF sends RS packets randomly, the tests for `testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy]` and `testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy]` fail intermittently.
Please enter the commit message for your changes. Lines starting
@JibinBao JibinBao force-pushed the qos_disable_ipv6_traffic branch from ea5ea4d to 2378480 Compare September 17, 2025 02:21
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@roy-sror roy-sror merged commit 9a35ada into sonic-net:master Oct 28, 2025
16 checks passed
albertovillarreal-keys pushed a commit to albertovillarreal-keys/sonic-mgmt that referenced this pull request Dec 2, 2025
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: Alberto Villarreal <alberto.villarreal@keysight.com>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Dec 11, 2025
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
saravanan-nexthop pushed a commit to saravanan-nexthop/sonic-mgmt that referenced this pull request Dec 15, 2025
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: Saravanan <saravanan@nexthop.ai>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Jan 29, 2026
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: Yael Tzur <ytzur@nvidia.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…D automatically (sonic-net#20416)

#### Why I did it
src/sonic-platform-daemons
```
* 10ef649 - (HEAD -> 202311, origin/202311) Improve parsing of media-settings.json for non-CMIS and breakout ports (sonic-net#533) (sonic-net#545) (9 hours ago) [longhuan-cisco]
```
#### How I did it
#### How to verify it
#### Description for the changelog
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Apr 20, 2026
…l introduce noise (sonic-net#20416)

When IPv6 is enabled on the PTF, PTF will send RS packets with DSCP 0, which corresponds to PG 1. Consequently, if we test PG 0, the PG watermark is affected because some part of the egress port buffer is occupied for TC 1. Since the PTF sends RS packets randomly, the tests for testQosSaiPgSharedWatermark[single_asic-wm_pg_shared_lossy] and testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy] fail intermittently.
To stabilize the cases, we disable the ipv6 traffic from ptf to dut for the two cases.

Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.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.

4 participants