-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(anta): Added test case to verify Spanning Tree (RPVST) state(stable topology) #791
feat(anta): Added test case to verify Spanning Tree (RPVST) state(stable topology) #791
Conversation
09e6496
to
ba29a99
Compare
e7ac1c2
to
a863d41
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add unit tests with outputs from different configured spanning-tree modes (mstp, rapid-pvst, rstp, none)?
DC1-LEAF1A(config)#spanning-tree mode ?
backup Backup port mode
mstp Multiple spanning tree protocol
none Disable spanning tree
rapid-pvst Per VLAN rapid spanning tree protocol
rstp Rapid spanning tree protocol
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
6cacf50
to
d84bbe8
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
Added unit test for rstp mode. |
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Verifies the number of changes across all interfaces in the Spanning Tree Protocol (STP) topology is below a threshold.
Fixes #782
Checklist:
pre-commit run
)tox -e testenv
)