Add comprehensive PVST test plan document#3
Open
devin-ai-integration[bot] wants to merge 1 commit into
Open
Conversation
This document provides a test plan for validating Per-VLAN Spanning Tree (PVST) functionality in SONiC using the T0 topology configurations. Includes: - Overview of PVST protocol and variants (PVST/RPVST/MSTP) - Test environment documentation for two_vlan_a and four_vlan_a topologies - 21 comprehensive test cases covering: - Per-VLAN STP instance independence - Interface-specific PVST configuration - Topology change resilience - Protocol variant testing - Configuration persistence - API reference for spytest/apis/switching/pvst.py functions - Validation criteria and pass/fail definitions Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
11 tasks
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
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 a comprehensive test plan document for validating Per-VLAN Spanning Tree (PVST) functionality in SONiC. The test plan covers PVST and Rapid PVST (RPVST) protocol implementations using the T0 topology configurations with two VLAN and four VLAN setups.
The document includes:
two_vlan_aandfour_vlan_atopologies fromansible/vars/topo_t0-isolated-d16u16s1.ymlspytest/apis/switching/pvst.pyReviewer focus areas:
spytest/apis/switching/pvst.pyansible/vars/topo_t0-isolated-d16u16s1.ymlType of change
Back port request
Approach
What is the motivation for this PR?
To provide a structured test plan for validating PVST functionality in SONiC, enabling systematic testing of per-VLAN spanning tree instances using the existing SPyTest PVST API.
How did you do it?
Created a comprehensive markdown document that:
spytest/apis/switching/pvst.pyHow did you verify/test it?
This is a documentation-only change. The test plan document was reviewed for:
spytest/apis/switching/pvst.pyansible/vars/topo_t0-isolated-d16u16s1.ymlAny platform specific information?
N/A - This is a test plan document applicable to all platforms supporting PVST.
Supported testbed topology if it's a new test case?
The test plan targets T0 topology with
two_vlan_aandfour_vlan_aconfigurations.Documentation
This PR itself is documentation - a test plan for PVST functionality.
Link to Devin run: https://cisco-demo.devinenterprise.com/sessions/bdf0af5058584cdfa8d54c0d72cd2882
Requested by: Arthur Poon (arthur.poon@cognition.ai) / @arthurkkp-cog