Skip to content

[db_migrator] Add version_202411_02 #3864

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benle7
Copy link

@benle7 benle7 commented Apr 24, 2025

What I did

Fixed migration error during upgrade from 202411_02.

How I did it

  • Added version_202411_02 function in db_migrator to support proper migration during upgrade
    (called dynamically via getattr from the migrate function).
  • Updated db_migrator_test accordingly.

How to verify it

Manual:

  • Use canonical setup with 202411 image
  • Verify the version in config_db and Redis is version_202411_02
    hget VERSIONS|DATABASE VERSION
    vim /etc/sonic/config_db.json
  • Install master image and reboot
  • Check syslog for db_migrator errors or run config reload

Automatic:

  • Use canonical setup with 202411 image
  • Verify the version in config_db and Redis is version_202411_02
  • Run test_deploy-and-upgrade

Additional Option:

  • Verify the version in config_db and Redis is version_202411_02
  • Run the following command to migrate (to 202505_01):
    /usr/local/bin/db_migrator.py -o migrate

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

Copy link

linux-foundation-easycla bot commented Apr 24, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: benle7 / name: Ben Levi (a222410)

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@benle7 benle7 force-pushed the benle/fix/db_migrator branch from 8bd7d20 to a222410 Compare April 24, 2025 12:17
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@benle7 benle7 changed the title Add version_202411_02 function [db_migrator] Add version_202411_02 function Apr 24, 2025
@benle7 benle7 changed the title [db_migrator] Add version_202411_02 function [db_migrator] Add version_202411_02 Apr 24, 2025
@benle7
Copy link
Author

benle7 commented Apr 29, 2025

@qiluo-msft

@benle7
Copy link
Author

benle7 commented Apr 30, 2025

@ryanzhu706 @vaibhavhd

@keboliu
Copy link
Collaborator

keboliu commented May 8, 2025

Hi @ryanzhu706 @vaibhavhd would you please help to review?

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.

4 participants