Adding new Tests for Chassis/Multi-ASIC GCU #14887
Merged
arlakshm merged 26 commits intosonic-net:masterfrom Jan 8, 2026
Merged
Adding new Tests for Chassis/Multi-ASIC GCU #14887arlakshm merged 26 commits intosonic-net:masterfrom
arlakshm merged 26 commits intosonic-net:masterfrom
Conversation
8 tasks
judyjoseph
reviewed
Feb 2, 2025
judyjoseph
reviewed
Feb 2, 2025
judyjoseph
reviewed
Feb 2, 2025
11 tasks
Collaborator
|
/azp run |
|
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. |
Collaborator
|
/azp run |
|
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. |
Collaborator
|
/azp run |
|
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. |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
|
The sonic-mgmt tests fail, @okaravasi could you rebase this branch ? |
okaravasi
force-pushed
the
master-new_tests_for_multi_asic_gcu
branch
from
0a028fe to
2cd5e41
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
vrajeshe
pushed a commit
to Akshath-17/sonic-mgmt
that referenced
this pull request
Jan 4, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887). Signed-off-by: Venkata Gouri Rajesh Etla <[email protected]>
arlakshm
approved these changes
Jan 8, 2026
arlakshm
pushed a commit
that referenced
this pull request
Jan 13, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan #21388. This PR has dependency on #14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test.
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Jan 13, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887).
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Jan 13, 2026
Description of PR Summary: This PR adds Multi-ASIC new tests for generic config updater (GCU). The changes of this PR have dependency on below PRs: sonic-net#15182 - merged sonic-net#20913 - open sonic-net#20914 - merged Type of change Bug fix Testbed and Framework(new/improvement) [X ] Test case(new/improvement) Back port request 202012 202205 202305 202311 [X ] 202405 Approach What is the motivation for this PR? To add new tests for Multi-ASIC/Chassis test support for GCU functionality. How did you do it? How did you verify/test it? Verified new tests in t2 testbeds.
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Jan 13, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan sonic-net#21388. This PR has dependency on sonic-net#14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test.
yifan-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Jan 14, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887). Signed-off-by: YiFan Wang <[email protected]>
yifan-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Jan 14, 2026
Description of PR Summary: This PR adds Multi-ASIC new tests for generic config updater (GCU). The changes of this PR have dependency on below PRs: sonic-net#15182 - merged sonic-net#20913 - open sonic-net#20914 - merged Type of change Bug fix Testbed and Framework(new/improvement) [X ] Test case(new/improvement) Back port request 202012 202205 202305 202311 [X ] 202405 Approach What is the motivation for this PR? To add new tests for Multi-ASIC/Chassis test support for GCU functionality. How did you do it? How did you verify/test it? Verified new tests in t2 testbeds. Signed-off-by: YiFan Wang <[email protected]>
yifan-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Jan 14, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan sonic-net#21388. This PR has dependency on sonic-net#14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test. Signed-off-by: YiFan Wang <[email protected]>
PriyanshTratiya
pushed a commit
to PriyanshTratiya/sonic-mgmt
that referenced
this pull request
Jan 21, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887). Signed-off-by: Priyansh Tratiya <[email protected]>
PriyanshTratiya
pushed a commit
to PriyanshTratiya/sonic-mgmt
that referenced
this pull request
Jan 21, 2026
Description of PR Summary: This PR adds Multi-ASIC new tests for generic config updater (GCU). The changes of this PR have dependency on below PRs: sonic-net#15182 - merged sonic-net#20913 - open sonic-net#20914 - merged Type of change Bug fix Testbed and Framework(new/improvement) [X ] Test case(new/improvement) Back port request 202012 202205 202305 202311 [X ] 202405 Approach What is the motivation for this PR? To add new tests for Multi-ASIC/Chassis test support for GCU functionality. How did you do it? How did you verify/test it? Verified new tests in t2 testbeds. Signed-off-by: Priyansh Tratiya <[email protected]>
PriyanshTratiya
pushed a commit
to PriyanshTratiya/sonic-mgmt
that referenced
this pull request
Jan 21, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan sonic-net#21388. This PR has dependency on sonic-net#14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test. Signed-off-by: Priyansh Tratiya <[email protected]>
AndoniSanguesa
pushed a commit
to AndoniSanguesa/sonic-mgmt
that referenced
this pull request
Jan 21, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan sonic-net#21388. This PR has dependency on sonic-net#14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test. Signed-off-by: Andoni Sanguesa <[email protected]>
AndoniSanguesa
pushed a commit
to AndoniSanguesa/sonic-mgmt
that referenced
this pull request
Jan 21, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan sonic-net#21388. This PR has dependency on sonic-net#14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test. Signed-off-by: Andoni Sanguesa <[email protected]>
lakshmi-nexthop
pushed a commit
to lakshmi-nexthop/sonic-mgmt
that referenced
this pull request
Jan 28, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887). Signed-off-by: Lakshmi Yarramaneni <[email protected]>
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Jan 29, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887).
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887). Signed-off-by: Yael Tzur <[email protected]>
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
Description of PR Summary: This PR adds Multi-ASIC new tests for generic config updater (GCU). The changes of this PR have dependency on below PRs: sonic-net#15182 - merged sonic-net#20913 - open sonic-net#20914 - merged Type of change Bug fix Testbed and Framework(new/improvement) [X ] Test case(new/improvement) Back port request 202012 202205 202305 202311 [X ] 202405 Approach What is the motivation for this PR? To add new tests for Multi-ASIC/Chassis test support for GCU functionality. How did you do it? How did you verify/test it? Verified new tests in t2 testbeds. Signed-off-by: Yael Tzur <[email protected]>
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan sonic-net#21388. This PR has dependency on sonic-net#14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test. Signed-off-by: Yael Tzur <[email protected]>
abhishek-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Feb 6, 2026
Adding fixture that selects a downstream duthost. Repo had already fixture for selecting upstream duthost from the available frontend hosts. This is to cover the gap and add the similar fixture for the downstream type. This is used by some suites, especially in t2 (e.g. gcu add cluster case sonic-net#14887).
abhishek-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Feb 6, 2026
Description of PR Summary: This PR adds Multi-ASIC new tests for generic config updater (GCU). The changes of this PR have dependency on below PRs: sonic-net#15182 - merged sonic-net#20913 - open sonic-net#20914 - merged Type of change Bug fix Testbed and Framework(new/improvement) [X ] Test case(new/improvement) Back port request 202012 202205 202305 202311 [X ] 202405 Approach What is the motivation for this PR? To add new tests for Multi-ASIC/Chassis test support for GCU functionality. How did you do it? How did you verify/test it? Verified new tests in t2 testbeds.
abhishek-nexthop
pushed a commit
to nexthop-ai/sonic-mgmt
that referenced
this pull request
Feb 6, 2026
Description of PR Summary: Fixes # (issue) Fixes # (issue) This PR implements the test case described in this test plan sonic-net#21388. This PR has dependency on sonic-net#14887 as it imports and uses functions that are added there. Type of change Bug fix Testbed and Framework(new/improvement) [ x] New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 [x ] 202405 202411 202505 [ x] 202511 Approach What is the motivation for this PR? To add new test case for gcu port speed change. How did you do it? How did you verify/test it? Ran this test again t2 voq chassis Any platform specific information? Platform constants file data is used by test to determine supported speeds which is require for the test. Added Nokia specific values, other vendors will have to add data applicable to their platform to run this test.
nnelluri-cisco
pushed a commit
to nnelluri-cisco/sonic-mgmt
that referenced
this pull request
Feb 12, 2026
Description of PR Summary: This PR adds Multi-ASIC new tests for generic config updater (GCU). The changes of this PR have dependency on below PRs: sonic-net#15182 - merged sonic-net#20913 - open sonic-net#20914 - merged Type of change Bug fix Testbed and Framework(new/improvement) [X ] Test case(new/improvement) Back port request 202012 202205 202305 202311 [X ] 202405 Approach What is the motivation for this PR? To add new tests for Multi-ASIC/Chassis test support for GCU functionality. How did you do it? How did you verify/test it? Verified new tests in t2 testbeds. Signed-off-by: nnelluri-cisco <[email protected]>
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
12 participants
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:
This PR adds Multi-ASIC new tests for generic config updater (GCU).
The changes of this PR have dependency on below PRs:
#15182 - merged
#20913 - open
#20914 - merged
Type of change
Back port request
Approach
What is the motivation for this PR?
To add new tests for Multi-ASIC/Chassis test support for GCU functionality.
How did you do it?
How did you verify/test it?
Verified new tests in t2 testbeds.
Attached the basic add cluster case with acl rules configured.
add_cluster.zip
Any platform specific information?
t2 testbeds
Supported testbed topology if it's a new test case?
t2
Documentation