Run db_migrator for non first-time reboots #71
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
dgsudharsan
approved these changes
Aug 11, 2023
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
vivekrnv
pushed a commit
that referenced
this pull request
Oct 13, 2023
…atically (sonic-net#15544) src/sonic-telemetry * 0b8843c - (HEAD -> 202205, origin/202205) Merge pull request sonic-net#120 from zbud-msft/202205_divide_by_zero (3 hours ago) [Ying Xie] | * 015defa - Merge branch '202205' into 202205_divide_by_zero (5 hours ago) [Zain Budhwani] * | de2124b - Change log level (5 hours ago) [zbud-msft] * | f203be5 - Add logs for md5 checksum (#80) (5 hours ago) [Zain Budhwani] | * ea6c84b - Merge branch '202205' into 202205_divide_by_zero (31 hours ago) [Zain Budhwani] * | ab98380 - Fix sonic-mgmt-common version to ec32690 in pipeline (sonic-net#123) (34 hours ago) [Sachin Holla] * 5fcecef - Merge branch '202205' into 202205_divide_by_zero (4 days ago) [Ying Xie] * 09c8bfc - Merge branch '202205' into 202205_divide_by_zero (11 days ago) [Zain Budhwani] * 21b9bc8 - Fix crash when retrieving cpu utilization (#70) (#71) (11 days ago) [Zain Budhwani]
vivekrnv
pushed a commit
that referenced
this pull request
Dec 18, 2024
…ically (sonic-net#21065) #### Why I did it src/sonic-dbsyncd ``` * b0ea01f - (HEAD -> master, origin/test_202411_20, origin/test_202411_100, origin/master, origin/kperumal_202411, origin/HEAD, origin/202411) [lldp]Fix the issue of only one field lldp_rem_time_mark in APPL_DB (#71) (2 days ago) [Zhaohui Sun] ``` #### How I did it #### How to verify it #### Description for the changelog
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why I did it
The recent change
https://github.com/sonic-net/sonic-buildimage/pull/15685#discussion_r1268400869removed 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
Work item tracking
How I did it
Re-introduce the logic to run the db_migrator on non-first boots
How to verify it
Which release branch to backport (provide reason below if selected)
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)