Skip to content

reliable tsa tests are now skipped on single asic voq duts#22089

Merged
saravanan-nexthop merged 4 commits intosonic-net:masterfrom
saravanan-nexthop:saravanan.voqtsa
Mar 18, 2026
Merged

reliable tsa tests are now skipped on single asic voq duts#22089
saravanan-nexthop merged 4 commits intosonic-net:masterfrom
saravanan-nexthop:saravanan.voqtsa

Conversation

@saravanan-nexthop
Copy link
Contributor

reliable tsa tests require two duthosts: one as supe and one as LC single asic voq dut have only one duthost, hence reliable tsa tests are skipped

Description of PR

Summary:
Fixes # (issue)
tests/bgp/reliable_tsa tests running failing on single asic voq duthosts

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?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

reliable tsa tests require two duthosts: one as supe and one as LC
single asic voq dut have only one duthost, hence reliable tsa tests are skipped

Signed-off-by: Saravanan Sellappa <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Contributor

arlakshm commented Feb 4, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm arlakshm added the Request for 202511 branch Request to backport a change to 202511 branch label Feb 4, 2026
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@arlakshm
Copy link
Contributor

arlakshm commented Mar 4, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@saravanan-nexthop saravanan-nexthop merged commit 18fdb2b into sonic-net:master Mar 18, 2026
16 checks passed
@mssonicbld
Copy link
Collaborator

@saravanan-nexthop PR conflicts with 202511 branch

saravanan-nexthop added a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 19, 2026
…voq duts

reliable tsa tests require two duthosts: one as supe and one as LC single asic voq dut have only one duthost, hence reliable tsa tests are skipped
backporting sonic-net#22089

Signed-off-by: Saravanan Sellappa <[email protected]>
saravanan-nexthop added a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 19, 2026
…voq duts

reliable tsa tests require two duthosts: one as supe and one as LC single asic voq dut have only one duthost, hence reliable tsa tests are skipped
backporting sonic-net#22089

Signed-off-by: Saravanan Sellappa <[email protected]>
saravanan-nexthop added a commit to saravanan-nexthop/sonic-mgmt that referenced this pull request Mar 19, 2026
…voq duts

reliable tsa tests require two duthosts: one as supe and one as LC single asic voq dut have only one duthost, hence reliable tsa tests are skipped
backporting sonic-net#22089

Signed-off-by: Saravanan Sellappa <[email protected]>
vmittal-msft pushed a commit that referenced this pull request Mar 19, 2026
…voq duts (#23124)

reliable tsa tests require two duthosts: one as supe and one as LC single asic voq dut have only one duthost, hence reliable tsa tests are skipped
backporting #22089

Signed-off-by: Saravanan Sellappa <[email protected]>
vrajeshe pushed a commit to vrajeshe/sonic-mgmt that referenced this pull request Mar 23, 2026
…#22089)

reliable tsa tests require two duthosts: one as supe and one as LC
single asic voq dut have only one duthost, hence reliable tsa tests are skipped

Signed-off-by: Saravanan Sellappa <[email protected]>
Signed-off-by: Venkata Gouri Rajesh Etla <[email protected]>
vmittal-msft pushed a commit that referenced this pull request Mar 26, 2026
* Strict priority QOS test. (#21051)

* Strict priority QOS test.

Signed-off-by: Abhishek <[email protected]>

* Add q3d and portchannel support for strict priority.

Signed-off-by: Abhishek <[email protected]>

* Fix per-commit and missing function.

Signed-off-by: Abhishek <[email protected]>

---------

Signed-off-by: Abhishek <[email protected]>

* [backport][202511] reliable tsa tests are now skipped on single asic voq duts

reliable tsa tests require two duthosts: one as supe and one as LC single asic voq dut have only one duthost, hence reliable tsa tests are skipped
backporting #22089

Signed-off-by: Saravanan Sellappa <[email protected]>

* Revert "Strict priority QOS test. (#21051)"

This reverts commit 88c7897.

Signed-off-by: venu-nexthop <[email protected]>

* Revert "[backport][202511] reliable tsa tests are now skipped on single asic voq duts"

This reverts commit 66748e6.

Signed-off-by: venu-nexthop <[email protected]>

* Back porting #22224 to 202511

Signed-off-by: venu-nexthop <[email protected]>

---------

Signed-off-by: Abhishek <[email protected]>
Signed-off-by: Saravanan Sellappa <[email protected]>
Signed-off-by: venu-nexthop <[email protected]>
Co-authored-by: abhishek-nexthop <[email protected]>
Co-authored-by: saravanan sellappa <[email protected]>
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
…#22089)

reliable tsa tests require two duthosts: one as supe and one as LC
single asic voq dut have only one duthost, hence reliable tsa tests are skipped

Signed-off-by: Saravanan Sellappa <[email protected]>
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.

5 participants