Skip to content

[202211][config reload]Config Reload Enhancement #15334

Merged
StormLiangMS merged 2 commits intosonic-net:202211from
dgsudharsan:cfg_rel_2211
Jun 12, 2023
Merged

[202211][config reload]Config Reload Enhancement #15334
StormLiangMS merged 2 commits intosonic-net:202211from
dgsudharsan:cfg_rel_2211

Conversation

@dgsudharsan
Copy link
Copy Markdown
Collaborator

Backporting #13969

Why I did it

Implementing code changes for sonic-net/SONiC#1203

Work item tracking
  • Microsoft ADO (number only):

How I did it

Removed the timers and delayed target since the delayed services would start based on event driven approach.
Cleared port table during config reload and cold reboot scenario.
Modified yang model, init_cfg.json to change has_timer to delayed

How to verify it

Added UT to verify

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

dgsudharsan and others added 2 commits June 10, 2023 02:35
Implementing code changes for sonic-net/SONiC#1203

Removed the timers and delayed target since the delayed services would start based on event driven approach.
Cleared port table during config reload and cold reboot scenario.
Modified yang model, init_cfg.json to change has_timer to delayed

Running regression
@dgsudharsan
Copy link
Copy Markdown
Collaborator Author

@StormLiangMS Can you please merge this?

Copy link
Copy Markdown
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

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