PVST Feature commit #20417
PVST Feature commit #20417
Conversation
divyachandralekha
requested review from
qiluo-msft,
xumia and
lguohan
as code owners
|
/azpw run Azure.sonic-buildimage |
|
/AzurePipelines run Azure.sonic-buildimage |
|
Commenter does not have sufficient privileges for PR 20417 in repo sonic-net/sonic-buildimage |
|
/azpw run Azure.sonic-buildimage |
|
/AzurePipelines run Azure.sonic-buildimage |
|
Pull request contains merge conflicts. |
|
/azpw ms_conflict |
2 similar comments
|
/azpw ms_conflict |
|
/azpw ms_conflict |
|
/AzurePipelines run ms_conflict |
|
Pull request contains merge conflicts. |
|
Pull request contains merge conflicts. |
.gitmodules
Outdated
| @@ -9,7 +9,8 @@ | |||
| url = https://github.com/sonic-net/sonic-sairedis | |||
| [submodule "sonic-swss"] | |||
| path = src/sonic-swss | |||
| url = https://github.com/sonic-net/sonic-swss | |||
| url = https://github.com/divyachandralekha/sonic-swss | |||
| branch = pvst_co_pr | |||
There was a problem hiding this comment.
You plan to integrate your code into sonic-net/sonic-swss right?
.gitmodules
Outdated
| @@ -30,7 +31,8 @@ | |||
| url = https://github.com/p4lang/ptf.git | |||
| [submodule "src/sonic-utilities"] | |||
| path = src/sonic-utilities | |||
| url = https://github.com/sonic-net/sonic-utilities | |||
| url = https://github.com/divyachandralekha/sonic-utilities | |||
| branch = pvst_co_pr | |||
There was a problem hiding this comment.
You plan to integrate your code into sonic-net/sonic-utilities right?
.gitmodules
Outdated
| url = https://github.com/Marvell-switching/sonic-platform-marvell | ||
| [submodule "src/sonic-stp"] | ||
| path = src/sonic-stp | ||
| url = https://github.com/divyachandralekha/sonic-stp |
There was a problem hiding this comment.
You plan to integrate your code into sonic-net/sonic-stp right?
Co-authored-by: Mati Alfaro <[email protected]>
Co-authored-by: Mati Alfaro <[email protected]>
Co-authored-by: Mati Alfaro <[email protected]>
|
It's good to attach the test results and test cases list. |
|
sonic-stp , sonic-utilities , sonic-swss should be merged first to resolve the failures in this repo. |
|
@divyachandralekha @ridahanif96 Kindly update if this PR is being actively worked upon and what would be the ETA for the commit of these changes. |
divyachandralekha
force-pushed
the
pvst_co_pr
branch
from
c404366 to
68926e6
Compare
|
/azpw run |
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
What I did
Added new STP docker and rules to compile STP
Added libevent required for STP
How to verify it
After this PR STP docker will be available