Remove the `force_to_redownload` column on the registry tables
requested to merge 417474-geo-remove-the-force_to_redownload-column-on-the-registry-tables into master
What does this MR do and why?
Several releases ago from last year, we deprecated the column named #force_to_redownload
in these five tables:
design_management_repository_registry
group_wiki_repository_registry
project_repository_registry
project_wiki_repository_registry
snippet_repository_registry
The column was marked with ignore_column
in the model definition per the process for avoiding downtime from removing columns. This MR updates the annotations for the current release version and date in the ignore_column
definitions, and then establishes Geo DB migrations to drop the column from each of these tables.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
How to set up and validate locally
- Run migrations on your geo secondary site up and down to verify they execute smoothly.
Questions for Reviewers
Since this is my first time pushing a migration to the Geo DB, I have a few questions for my reviewers:
- Do I have to run the migrations and commit changes to my
ee/db/geo/structure.sql
? For some reason it seemed to radically change it on my dev environment relative to what's checked into source control. - Is there danger in changing the
ignore_column
milestone and date parameters to the current 16.9 release? And am I incorrect already ... should it now be 16.10 and next month's date?
Related to #417474 (closed)
Edited by Kyle Yetter