Skip to content

Draft: Remove post-deployment migration 20210622041846 and migrate back for GitLab.com

What does this MR do?

We found that the Database::BatchedBackgroundMigrationWorker is running only for GitLab.com. This mean that even if we though we shipped PK conversion batched migrations for self-managed, they are not being processed, and self-managed instances are not ready to finalize PK conversions in 14.1 - ensure_batched_background_migration_is_finished will be failing for them.

This MR is fixing this problem by building on top of !65105 (closed) (removing the migration db/post_migrate/20210622041846_finalize_push_event_payloads_bigint_conversion.rb) by adding GitLab.com only migration that will migrate push_event_payloads.eevent_id back to integer. This will keep the schema the same for GitLab.com and self-managed.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #288005 (closed)

Edited by Krasimir Angelov

Merge request reports