Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

migrator: Run oobmigrations in upgrade command#40628

Merged
efritz merged 2 commits into
mainfrom
ef/oobmigrations-upgrade
Aug 20, 2022
Merged

migrator: Run oobmigrations in upgrade command#40628
efritz merged 2 commits into
mainfrom
ef/oobmigrations-upgrade

Conversation

@efritz

@efritz efritz commented Aug 19, 2022

Copy link
Copy Markdown
Contributor

This PR implements the missing functionality of running oobmigrations in the migrator upgrade command. Fixes #39578.

Test plan

Tested locally.

@cla-bot cla-bot Bot added the cla-signed label Aug 19, 2022
Base automatically changed from ef/run-oobmigrations-in-migrator to main August 19, 2022 22:18
@efritz efritz force-pushed the ef/oobmigrations-upgrade branch from a53ebd7 to 00e7a68 Compare August 19, 2022 22:36
@efritz efritz self-assigned this Aug 20, 2022
@efritz efritz force-pushed the ef/oobmigrations-upgrade branch from 3209447 to 918794b Compare August 20, 2022 17:22
@efritz efritz force-pushed the ef/oobmigrations-upgrade branch from 9a9c2f9 to 62500f9 Compare August 20, 2022 21:03
@efritz efritz requested review from emidoots and eseliger August 20, 2022 21:04
@efritz efritz changed the title WIP. migrator: Run oobmigrations in upgrade command Aug 20, 2022
@efritz efritz marked this pull request as ready for review August 20, 2022 21:05
@efritz efritz merged commit 12b8503 into main Aug 20, 2022
@efritz efritz deleted the ef/oobmigrations-upgrade branch August 20, 2022 21:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RFC 697: Run data migrations over multiple versions in the migrator

2 participants