Skip to content

Recreate tmp_idx_vulns_on_converted_uuid synchronously

Michał Zając requested to merge mzajac/recreate-proper-index-after-incident-15983 into master

What does this MR do and why?

Create type migration supporting index synchron... (!125219 - merged) tried to create the index using the wrong column (uuid instead of uuid_convert_string_to_uuid) which caused gitlab-com/gl-infra/production#15983 (closed).

We made the index creation a no-op in Make wrong index creation migration a no-op (!125422 - merged) but anyone who already created the index will have the incorrect index

This MR:

  1. Drops the tmp_idx_vulns_on_converted_uuid index UNLESS it's ran on Gitlab.com, on Gitlab.com we created the index asynchronously over the weekend so it's the correct one
  2. Recreates tmp_idx_vulns_on_converted_uuid using uuid_convert_string_to_uuid column

MR acceptance checklist

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

Merge request reports