Skip to content

elasticsearch: Fix indexing blobs and commits with strict mapping on

What does this MR do?

The elasticsearch index recently gained strict, rather than dynamic, mapping. However, we're still missing two important fields:

  • repository.commit.type
  • repository.blob.type

I mistakenly thought all was well, when in fact our Ruby and Go indexers both silently ignore documents that fail to create. gitlab-elasticsearch-indexer#28 (closed) is a follow-up issue for that problem.

This fix is sufficient to bring the document count back up to the right number on my test instance (~262,000 instead of ~2000)

What are the relevant issue numbers?

Regression introduced in https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/9760

Does this MR meet the acceptance criteria?

cc @mdelaossa

Edited by Nick Thomas

Merge request reports