Skip to content

[202411] Dualtor AA enablement for QOS SAI#17911

Merged
bpar9 merged 2 commits intosonic-net:202411from
rbpittman:qos_sai_dualtor_aa_202411
Jun 13, 2025
Merged

[202411] Dualtor AA enablement for QOS SAI#17911
bpar9 merged 2 commits intosonic-net:202411from
rbpittman:qos_sai_dualtor_aa_202411

Conversation

@rbpittman
Copy link
Contributor

@rbpittman rbpittman commented Apr 9, 2025

Description of PR

Summary:
Double commit of #17765

  • Update test mark conditions to include dualtor-aa for every duplicate instance of the QOS SAI topo list (all lists are identical). Possible de-duplication here: Tests mark constants #17707
  • Only toggle simulator ports if active-standby.
  • Fix permissions on created empty write_standby file.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Master PR: Ran on dualtor AA cisco-8000, verified tests execute without major errors. Some tests fail due to some normal instability and known issues.

Double commit:

  • Perform same validation as on master

Any platform specific information?

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

Documentation

* Apply diff for dualtor AA QOS SAI with list change duplicated for all qos sai topo lists.

* Add back a newline.
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rbpittman rbpittman mentioned this pull request Apr 9, 2025
11 tasks
@lolyu
Copy link
Collaborator

lolyu commented Apr 27, 2025

Hi @rbpittman, please help resolve the conflict.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rbpittman
Copy link
Contributor Author

@kevinskwang
Can you help merge?

@bpar9 bpar9 merged commit 194b907 into sonic-net:202411 Jun 13, 2025
14 checks passed
@rbpittman rbpittman deleted the qos_sai_dualtor_aa_202411 branch June 13, 2025 13:35
sdszhang pushed a commit to sdszhang/sonic-mgmt that referenced this pull request Jun 30, 2025
Code sync sonic-net/sonic-mgmt:202411 => 202412

```
*   86163d9 (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250620:0224 - Merge remote-tracking branch 'base/202411' into code-sync-202412
|\
| * 6c7825f (base/202411) Justin Wong 250611:2109 - Skip test_bgp_multipath_relax.py on t1-isolated (sonic-net#18558)
| * 829aa40 Chun'ang Li 250619:1618 - [CI][202411]update pr test template reference (sonic-net#19059)
| * 6f2a1bd prabhataravind 250617:1800 - Fix port mapping for new 4280 hwskus (sonic-net#18802) (sonic-net#19030)
| * 66c7667 ganglv 250617:1623 - [202411] Update timeout for gnmi subscribe API (sonic-net#19045)
| * 8f0a97b Justin Wong 250611:2146 - Extend bgp command check to allow multipath routes and "best" routes (sonic-net#18910)
| * 12b7793 StormLiangMS 250611:1110 - [XFAIL: test_crm_available.py] set xfail of test_crm_available on dualtor platform (sonic-net#18874)
| * ab5fd70 rbpittman 250615:2148 - Shutdown BGP instead of stopping some processes. (sonic-net#18267)
| * 3f35fa2 Harish Kalyanaraman 250321:0332 - Increase GCUTIMEOUT value for nokia-armhf platforms (sonic-net#17420)
| * 48bf447 xwjiang-ms 250613:1238 - [202411] Support Ubuntu 24 server in KVM (sonic-net#18929)
| * 194b907 rbpittman 250612:2057 - Dualtor AA enablement for QOS SAI (sonic-net#17765) (sonic-net#17911)
```
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.

4 participants