Skip to content

PVSTP feature implementation#648

Open
ph408077 wants to merge 6 commits intosonic-net:masterfrom
ph408077:stp_pr
Open

PVSTP feature implementation#648
ph408077 wants to merge 6 commits intosonic-net:masterfrom
ph408077:stp_pr

Conversation

@ph408077
Copy link

  • Added config/show/debug/clear handling for STP
  • Modified other utilities script to include STP

- What I did

- How I did it

- How to verify it

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

-->

@ph408077
Copy link
Author

Not sure why 164 tests failed. Not able to access the Details link as well. Please let me know if anything needs to be done from my side.

mhampasagar
mhampasagar previously approved these changes Nov 27, 2019
- Added config/show/debug/clear handling for STP
- Modified other utilities script to include STP
@lgtm-com
Copy link

lgtm-com bot commented Apr 20, 2021

This pull request introduces 53 alerts when merging 2ab9746 into e296a69 - view on LGTM.com

new alerts:

  • 44 for Unnecessary pass
  • 4 for Unused import
  • 2 for Unreachable code
  • 2 for Module-level cyclic import
  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Apr 23, 2021

This pull request introduces 5 alerts when merging 76386b1 into b10c157 - view on LGTM.com

new alerts:

  • 3 for Module-level cyclic import
  • 2 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Apr 23, 2021

This pull request introduces 1 alert when merging 2de4355 into b10c157 - view on LGTM.com

new alerts:

  • 1 for Unused import

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.

3 participants