Skip to content

Conversation

@haojun291
Copy link

@haojun291 haojun291 commented Oct 31, 2025

[teamsyncd] Ignore LAG member port 'enable' message for down ports

What I did
 Added a check for the physical port's up/down state when teamd reports member port status. This ensures that the 'enable' message for a member port is ignored if the physical port is down.

Why I did it
 On platforms using the Trident4 ASIC, there is a specific issue in the SAI (Switch Abstraction Interface). Processing an 'enable' message for a port that is physically down causes an SAI error during operations like port splitting or removing a port from a LAG. This fix prevents that error from happening.

How I verified it
 Verified on a Trident4-based physical switch. I confirmed that removing a 'down' member port from a LAG no longer triggers the SAI error.

What I did
 Added a check for the physical port's up/down state when  teamd  reports member port status. This ensures that the 'enable' message for a member port is ignored if the physical port is down.

Why I did it
 On platforms using the Trident4 ASIC, there is a specific issue in the SAI (Switch Abstraction Interface). Processing an 'enable' message for a port that is physically down causes an SAI error during operations like port splitting or removing a port from a LAG. This fix prevents that error from happening.

How I verified it
 Verified on a Trident4-based physical switch. I confirmed that removing a 'down' member port from a LAG no longer triggers the SAI error.
@linux-foundation-easycla
Copy link

CLA Not Signed

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@haojun291 haojun291 changed the base branch from master to 202405 October 31, 2025 10:20
@haojun291 haojun291 changed the title Ignore member port enable message for down ports Ignore LAG member port enable message for down ports Nov 12, 2025
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.

2 participants