From 7dc4d0c72fee71738c878f6646907be7efeeab02 Mon Sep 17 00:00:00 2001 From: Vijay Hawoldar <vhawoldar@gitlab.com> Date: Fri, 1 Nov 2024 13:24:13 +0000 Subject: [PATCH] Add re-queuing note to batched migration deletion Updates the docs for deleting batched migrations to mention re-queuing. --- doc/development/database/batched_background_migrations.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/doc/development/database/batched_background_migrations.md b/doc/development/database/batched_background_migrations.md index 8215561aba36..7868753d89e2 100644 --- a/doc/development/database/batched_background_migrations.md +++ b/doc/development/database/batched_background_migrations.md @@ -307,8 +307,9 @@ migration code should be. ### Deleting batched background migration code -Once a batched background migration has been finalized, the migration code in `lib/gitlab/background_migration/` -and its associated tests can be deleted after the next required stop following the finalization. +Once a batched background migration has completed, is finalized and has not been [re-queued](#re-queue-batched-background-migrations), +the migration code in `lib/gitlab/background_migration/` and its associated tests can be deleted after the next required stop following +the finalization. Here is an example scenario: -- GitLab