Skip to content

Conversation

@Yogapriya-cisco
Copy link
Contributor

No description provided.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 13, 2025

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: skulambi-cisco / name: Sandeep Kulambi (fed4029)

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vrajeshe
Copy link

Hi @divyachandralekha this is to enable STP docker can you pls approve along with
sonic-net/sonic-buildimage#24521

case STP_BRIDGE_CONFIG:
{
if(msg->proto_mode == L2_PVSTP)
stpmgr_process_bridge_config_msg(msg->data);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert the changes

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@divyachandralekha, right now both PVST and MSTP are not working due to incorrect changes in stpmgrd. For now atleast this change will allow us to get PVST working with minimal changes. As part of MSTP we have to address the issue on stpmgrd where we pass the correct msg->proto_mode.

case STP_PORT_CONFIG:
{
if(msg->proto_mode == L2_PVSTP)
stpmgr_process_intf_config_msg(msg->data);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert the changes

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@divyachandralekha, right now both PVST and MSTP are not working due to incorrect changes in stpmgrd. For now atleast this change will allow us to get PVST working with minimal changes. As part of MSTP we have to address the issue on stpmgrd where we pass the correct msg->proto_mode.

@divyachandralekha
Copy link
Contributor

/azpw run Azure.sonic-stp

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-stp

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@divyachandralekha divyachandralekha merged commit d30e086 into sonic-net:master Nov 14, 2025
1 of 5 checks passed
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.

5 participants