Skip to content

Commit 2911707

Browse files
vivekrnvmssonicbld
authored andcommitted
Run db_migrator for non first-time reboots (sonic-net#16116)
- 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]>
1 parent dbfa8f9 commit 2911707

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

files/build_templates/docker_image_ctl.j2

+9
Original file line numberDiff line numberDiff line change
@@ -248,9 +248,18 @@ function postStartAction()
248248
# This flag will be set to "1" after DB migration/initialization is completed as part of config-setup
249249
$SONIC_DB_CLI CONFIG_DB SET "CONFIG_DB_INITIALIZED" "0"
250250
else
251+
$SONIC_DB_CLI CONFIG_DB SET "CONFIG_DB_INITIALIZED" "0"
252+
# this is not a first time boot to a new image. Datbase container starts w/ old pre-existing config
253+
if [[ -x /usr/local/bin/db_migrator.py ]]; then
254+
# Migrate the DB to the latest schema version if needed
255+
if [ -z "$DEV" ]; then
256+
/usr/local/bin/db_migrator.py -o migrate
257+
fi
258+
fi
251259
# set CONFIG_DB_INITIALIZED to indicate end of config load and migration
252260
$SONIC_DB_CLI CONFIG_DB SET "CONFIG_DB_INITIALIZED" "1"
253261
fi
262+
254263
# Add redis UDS to the redis group and give read/write access to the group
255264
REDIS_SOCK="/var/run/redis${DEV}/redis.sock"
256265
else

0 commit comments

Comments
 (0)