Skip to content

Remove old Advanced Search migrations

Dmitry Gruzd requested to merge 323310-remove-old-es-migrations into master

What does this MR do?

This change consists of 3 MRs:

This MR does multiple things:

  • Adds Elastic::MigrationObsolete as a way to deprecate Advanced Search migrations
  • Changes the set_elasticsearch_migration_to method to raise an exception if the migration is not found or it's obsolete
  • Deprecates the ApplyMaxAnalyzedOffset migration
  • Changes the documentation section Deleting Advanced Search migrations in a major version upgrade in order to reflect the changes in this MR

#323310 (closed)

Rollout plan

  1. Get approvals on all MRs (this MR, !63020 (merged), and !63021 (merged))
  2. Merge !63021 (merged) into !63020 (merged)
  3. Merge !63020 (merged) into this MR
  4. Merge this MR into master

Screenshots (strongly suggested)

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

Closes #323310 (closed)

Edited by Dmitry Gruzd

Merge request reports