diff options
author | Stan Hu <stanhu@gmail.com> | 2018-03-08 11:36:41 -0800 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2018-03-08 11:36:41 -0800 |
commit | 43957e7a8189dc23e5a1ad66a1a85255d65ed72c (patch) | |
tree | 0fe7422acd15efe26e4390f46bf34ab51a1360c9 /db/migrate | |
parent | 08c407cc0f9e52b6ac2aa2884bb055ccd15d70d7 (diff) | |
download | gitlab-ce-43957e7a8189dc23e5a1ad66a1a85255d65ed72c.tar.gz |
Fix index name to Rails default to ensure idempotencysh-make-add-index-idempotent
See https://gitlab.com/gitlab-com/infrastructure/issues/3822
Diffstat (limited to 'db/migrate')
-rw-r--r-- | db/migrate/20180308052825_add_section_name_id_index_on_ci_build_trace_sections.rb | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/db/migrate/20180308052825_add_section_name_id_index_on_ci_build_trace_sections.rb b/db/migrate/20180308052825_add_section_name_id_index_on_ci_build_trace_sections.rb index 0cf665ac935..b616cc2fd30 100644 --- a/db/migrate/20180308052825_add_section_name_id_index_on_ci_build_trace_sections.rb +++ b/db/migrate/20180308052825_add_section_name_id_index_on_ci_build_trace_sections.rb @@ -3,20 +3,21 @@ class AddSectionNameIdIndexOnCiBuildTraceSections < ActiveRecord::Migration # Set this constant to true if this migration requires downtime. DOWNTIME = false + INDEX_NAME = 'index_ci_build_trace_sections_on_section_name_id' disable_ddl_transaction! def up # MySQL may already have this as a foreign key - unless index_exists?(:ci_build_trace_sections, :section_name_id) - add_concurrent_index :ci_build_trace_sections, :section_name_id + unless index_exists?(:ci_build_trace_sections, :section_name_id, name: INDEX_NAME) + add_concurrent_index :ci_build_trace_sections, :section_name_id, name: INDEX_NAME end end def down # We cannot remove index for MySQL because it's needed for foreign key if Gitlab::Database.postgresql? - remove_concurrent_index :ci_build_trace_sections, :section_name_id + remove_concurrent_index :ci_build_trace_sections, :section_name_id, name: INDEX_NAME end end end |