Add mgmt test for testing fabric link isolation cli commands.#10994
Add mgmt test for testing fabric link isolation cli commands.#10994arlakshm merged 4 commits intosonic-net:masterfrom
Conversation
|
sonic-net/sonic-buildimage#17473 need to be merged first |
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
Signed-off-by: Jie Feng <[email protected]>
ab78d99 to
e38cda1
Compare
|
the failures do not seem related to the change |
|
/Azpw run Azure.sonic-mgmt |
|
/AzurePipelines run Azure.sonic-mgmt |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/Azpw run Azure.sonic-mgmt |
|
/AzurePipelines run Azure.sonic-mgmt |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@jfeng-arista please add more details in the PR description on new tests are added. |
arlakshm
left a comment
There was a problem hiding this comment.
test looks good. minor comments
revised. thank you ! |
Description of PR
Summary: Add mgmt test for testing fabric link isolation cli commands.
This change adds a test for the fabric link isolation command.
The test iterates over the fabric links on a linecard. It isolates and unisolates fabric links. Each time the state of the link is changed the state is checked in both CONFIG_DB and APPL_DB. The values in CONFIG_DB are updated by the fabric CLI commands and the values in APPL_DB are updated by the fabric manager daemon. The test check if the 2 major part work as expected.
need to wait sonic-net/sonic-buildimage#17473
Fixes # (issue)
Type of change
Back port request
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