Skip to content

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

Merged
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/16116
Aug 28, 2023
Merged

[action] [PR:16116] Run db_migrator for non first-time reboots#16306
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/16116

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 adfc486 into sonic-net:202305 Aug 28, 2023
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