Skip to content

[Warm-reboot] Skip lag_keepalive on inactive LAG members - #4748

Open
YairRaviv wants to merge 1 commit into
sonic-net:masterfrom
YairRaviv:yraviv-skip-lag-keepalive-for-inactive-lag-members
Open

[Warm-reboot] Skip lag_keepalive on inactive LAG members#4748
YairRaviv wants to merge 1 commit into
sonic-net:masterfrom
YairRaviv:yraviv-skip-lag-keepalive-for-inactive-lag-members

Conversation

@YairRaviv

@YairRaviv YairRaviv commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

What I did

Skip inactive LAG members from sending LACPDUs during fast/warm reboot

How I did it

By checking each LAG member admin status before crafting LACPDU packets

How to verify it

  1. Configure a portchannel with multiple members
  2. Shutdown 1 of the LAG members
  3. Run the lag_keepalive script and make sure LACPDUs are sent only for the active LAG members
  4. Verify no lag_keepalive errors

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)

Tested branch

  • master
  • 202605

Signed-off-by: Yair Raviv <yraviv@nvidia.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@YairRaviv YairRaviv changed the title [warm-reboot]Skip lag_keepalive on inactive LAG members [Warm-reboot] Skip lag_keepalive on inactive LAG members Aug 6, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@abelamit abelamit removed the Tested for 202605 branch Tested for 202605 branch label Aug 6, 2026
@mssonicbld

Copy link
Copy Markdown
Collaborator

This PR has backport request label(s) for branch(es): 202605, but is missing required test information. Please make sure you tick the tested branch(es) in the Tested branch section and provide test evidence (e.g., 202605: <test result>) in the Test result section as well in your PR description.

---Powered by SONiC BuildBot

@abelamit abelamit added the Tested for 202605 branch Tested for 202605 branch label Aug 8, 2026
@YairRaviv YairRaviv closed this Aug 9, 2026
@YairRaviv YairRaviv reopened this Aug 9, 2026
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@liat-grozovik

Copy link
Copy Markdown
Collaborator

@vaibhavhd could you please help review?

@YairRaviv YairRaviv closed this Aug 9, 2026
@YairRaviv YairRaviv reopened this Aug 9, 2026
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

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.

4 participants