Add and backfill project_id for approval_project_rules_protected_branche
## What does this MR do and why? Add and backfill project_id for approval_project_rules_protected_branches. This table has a [desired sharding key](https://docs.gitlab.com/ee/development/database/multiple_databases.html#define-a-desired_sharding_key-to-automatically-backfill-a-sharding_key) configured ([view configuration](https://gitlab.com/gitlab-org/gitlab/-/blob/master/db/docs/approval_project_rules_protected_branches.yml)). This merge request is the first step towards transforming the desired sharding key into a [sharding key](https://docs.gitlab.com/ee/development/database/multiple_databases.html#defining-a-sharding-key-for-all-cell-local-tables). This involves three changes: - Adding a new column that will serve as the sharding key (along with the relevant index and foreign key). - Populating the sharding key when new records are created by adding a database function and trigger. - Scheduling a [batched background migration](https://docs.gitlab.com/ee/development/database/batched_background_migrations.html) to set the sharding key for existing records. Once the background migration has completed, a second merge request will be created to finalize the background migration and validate the not null constraint. ## How to verify We have assigned a random backend engineer from ~"group::source code" to review these changes. Please review this merge request from a ~backend perspective. The main thing we are looking to verify is that the added column and association match the values specified by the [desired sharding key](https://gitlab.com/gitlab-org/gitlab/-/blob/master/db/docs/approval_project_rules_protected_branches.yml) configuration and that backfilling the column from this other table makes sense in the context of this feature. When you are finished, please: 1. Trigger the [database testing pipeline](https://docs.gitlab.com/ee/development/database/database_migration_pipeline.html) as instructed by Danger. 1. Request a review from the ~backend maintainer and ~database reviewer suggested by Danger. If you have any questions or concerns, reach out to `@tigerwnz` or @shubhamkrai. This merge request was generated by a once off keep implemented in https://gitlab.com/gitlab-org/gitlab/-/merge_requests/143774 This change was generated by [gitlab-housekeeper](https://gitlab.com/gitlab-org/gitlab/-/tree/master/gems/gitlab-housekeeper) using the Keeps::BackfillDesiredShardingKeySmallTable keep. To provide feedback on your experience with `gitlab-housekeeper` please create an issue with the label ~"GitLab Housekeeper" and consider pinging the author of this keep. Changelog: other
显示
- db/docs/approval_project_rules_protected_branches.yml 1 个添加, 0 个删除db/docs/approval_project_rules_protected_branches.yml
- db/docs/batched_background_migrations/backfill_approval_project_rules_protected_branches_project_id.yml 9 个添加, 0 个删除..._approval_project_rules_protected_branches_project_id.yml
- db/migrate/20240930160023_add_project_id_to_approval_project_rules_protected_branches.rb 9 个添加, 0 个删除...roject_id_to_approval_project_rules_protected_branches.rb
- db/post_migrate/20240930160024_index_approval_project_rules_protected_branches_on_project_id.rb 16 个添加, 0 个删除...pproval_project_rules_protected_branches_on_project_id.rb
- db/post_migrate/20240930160025_add_approval_project_rules_protected_branches_project_id_fk.rb 17 个添加, 0 个删除...pproval_project_rules_protected_branches_project_id_fk.rb
- db/post_migrate/20240930160026_add_approval_project_rules_protected_branches_project_id_trigger.rb 25 个添加, 0 个删除...al_project_rules_protected_branches_project_id_trigger.rb
- db/post_migrate/20240930160027_queue_backfill_approval_project_rules_protected_branches_project_id.rb 41 个添加, 0 个删除...l_approval_project_rules_protected_branches_project_id.rb
- db/schema_migrations/20240930160023 1 个添加, 0 个删除db/schema_migrations/20240930160023
- db/schema_migrations/20240930160024 1 个添加, 0 个删除db/schema_migrations/20240930160024
- db/schema_migrations/20240930160025 1 个添加, 0 个删除db/schema_migrations/20240930160025
- db/schema_migrations/20240930160026 1 个添加, 0 个删除db/schema_migrations/20240930160026
- db/schema_migrations/20240930160027 1 个添加, 0 个删除db/schema_migrations/20240930160027
- db/structure.sql 25 个添加, 1 个删除db/structure.sql
- lib/gitlab/background_migration/backfill_approval_project_rules_protected_branches_project_id.rb 10 个添加, 0 个删除...l_approval_project_rules_protected_branches_project_id.rb
- spec/lib/gitlab/background_migration/backfill_approval_project_rules_protected_branches_project_id_spec.rb 16 个添加, 0 个删除...roval_project_rules_protected_branches_project_id_spec.rb
- spec/migrations/20240930160027_queue_backfill_approval_project_rules_protected_branches_project_id_spec.rb 34 个添加, 0 个删除...roval_project_rules_protected_branches_project_id_spec.rb
加载中
想要评论请 注册 或 登录