Skip to content

Delete Project and related data in ES if use_elasticsearch? is false

Ravi Kumar requested to merge 410777-migration-reindex-project-wiki-to-fix-permissions-fix into master

What does this MR do and why?

  • This MR will call the ElasticDeleteProjectWorker from ElasticWikiIndexerWorker if the Project does not exist or .use_elasticsearch? is false. We should remove the project and associated ES documents if the Project is deleted or it is not using elasticsearch.

  • This MR will call the ElasticDeleteProjectWorker from ElasticCommitIndexerWorker if use_elasticsearch? is false for the Project.

  • Also removing the space_requirements from the migration file as it is not used.

  • If the ee/lib/gitlab/elastic/indexer.rb if the commit is not found, then we are calling purge_unreachable_commits_from_index!

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #410777

Edited by Ravi Kumar

Merge request reports