Skip to content
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

PVSTP feature implementation #3463

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ph408077
Copy link
Contributor

  • Added new STP docker and rules to compile STP
  • Added libevent required for STP

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

- Added new STP docker and rules to compile STP
- Added libevent required for STP
@ph408077
Copy link
Contributor Author

The build failure is expected as it is not able to find stp src location. Once sonic-stp repo is merged and build retriggered, there should be no build issues.

[ FAIL LOG START ] [ target/debs/stretch/stp_1.0.0_amd64.deb ]
/bin/bash: line 9: pushd: src/sonic-stp: No such file or directory
[ FAIL LOG END ] [ target/debs/stretch/stp_1.0.0_amd64.deb ]
slave.mk:335: recipe for target 'target/debs/stretch/stp_1.0.0_amd64.deb' failed
make: *** [target/debs/stretch/stp_1.0.0_amd64.deb] Error 1
Makefile.work:190: recipe for target 'target/sonic-broadcom.bin' failed
make[1]: *** [target/sonic-broadcom.bin] Error 2

nirenjan
nirenjan previously approved these changes Nov 12, 2019
@benedikt-bartscher
Copy link

Hey there, thanks for this great PR! Is this going to be merged any time soon?

@dcantunes123
Copy link

Trying to follow this up, is there an ETA for PVST to become available?

@dotie
Copy link

dotie commented Jan 24, 2023

@qiluo-msft @xumia @zhangyanzhao Are there any plans to approve PVSTP? This is supported in the Broadcom/Edge Core enterprise sonic builds.

@ITJamie
Copy link

ITJamie commented Feb 20, 2023

@gechiang @prgeor s there anychance someone could review this / take it over? its a really important feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants