Skip to content

[fast-reboot] Remove teamsyncd timer override by fast-boot#25266

Merged
lihuay merged 3 commits intosonic-net:masterfrom
YairRaviv:remove-teamsyncd-timer
Mar 9, 2026
Merged

[fast-reboot] Remove teamsyncd timer override by fast-boot#25266
lihuay merged 3 commits intosonic-net:masterfrom
YairRaviv:remove-teamsyncd-timer

Conversation

@YairRaviv
Copy link
Contributor

@YairRaviv YairRaviv commented Jan 29, 2026

This PR reopened this PR - #23445

What I did

Remove teamsyncd 1 sec timer override. It was used to speed up kernel IP configuration on PortChannel as a W/A.
Related to sonic-net/sonic-utilities#4233

Work item tracking
  • Microsoft ADO (number only):

How I did it

Remove teamsyncd 1 sec timer override.

How to verify it

Ran fast-boot and warm-boot tests.

Which release branch to backport (provide reason below if selected)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version)

  • 202505
  • master

Description for the changelog

Link to config_db schema for YANG module changes

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

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@YairRaviv YairRaviv requested a review from lguohan as a code owner January 29, 2026 15:00
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik
Copy link
Collaborator

@yxieca LGTM, if not objection lets merge this

@yxieca
Copy link
Contributor

yxieca commented Mar 5, 2026

@vaibhavhd please revew. Thanks.

@liat-grozovik
Copy link
Collaborator

@YairRaviv is this needed for master only or also for 202511?

@YairRaviv
Copy link
Contributor Author

YairRaviv commented Mar 9, 2026

@YairRaviv is this needed for master only or also for 202511?

@liat-grozovik master only

@lihuay lihuay merged commit e826821 into sonic-net:master Mar 9, 2026
26 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.

6 participants