Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[action] [PR:16116] Run db_migrator for non first-time reboots #16520

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

mssonicbld
Copy link
Collaborator

Why I did it

The recent change #15685 (comment) removed the db migration for non first reboots.

This is problematic for many deployments which doesn't rely on ZTP and push a custom config_db.json

Port to older branches after #15685 is ported back

Work item tracking
  • Microsoft ADO (number only):

How I did it

Re-introduce the logic to run the db_migrator on non-first boots

How to verify it

Verified reboot and warm-reboot cases

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

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

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)

- Why I did it
The recent change sonic-net#15685 (comment) removed the db migration for non first reboots.
This is problematic for many deployments which doesn't rely on ZTP and push a custom config_db.json
Port to older branches after sonic-net#15685 is ported back

- How I did it
Re-introduce the logic to run the db_migrator on non-first boots

- How to verify it
Verified reboot and warm-reboot cases

Signed-off-by: Vivek Reddy Karri <[email protected]>
@mssonicbld
Copy link
Collaborator Author

Original PR: #16116

@mssonicbld mssonicbld merged commit b4ab3e0 into sonic-net:202205 Sep 12, 2023
15 checks passed
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.

2 participants