make the db migration idempotent

db/migrate/20170825015534_add_file_store_to_lfs_objects.rb → ee/db/migrate/20170825015534_add_file_store_to_lfs_objects.rb

@@ -26,10 +26,14 @@ class AddFileStoreToLfsObjects < ActiveRecord::Migration
   disable_ddl_transaction!
 
   def up
-    add_column(:lfs_objects, :file_store, :integer)
+    unless column_exists?(:lfs_objects, :file_store)
+      add_column(:lfs_objects, :file_store, :integer)
+    end
   end
 
   def down
-    remove_column(:lfs_objects, :file_store)
+    if column_exists?(:lfs_objects, :file_store)
+      remove_column(:lfs_objects, :file_store)
+    end
   end
 end

db/migrate/20170918072949_add_file_store_job_artifacts.rb → ee/db/migrate/20170918072949_add_file_store_job_artifacts.rb

@@ -5,10 +5,14 @@ class AddFileStoreJobArtifacts < ActiveRecord::Migration
   DOWNTIME = false
 
   def up
-    add_column(:ci_job_artifacts, :file_store, :integer)
+    unless column_exists?(:ci_job_artifacts, :file_store)
+      add_column(:ci_job_artifacts, :file_store, :integer)
+    end
   end
 
   def down
-    remove_column(:ci_job_artifacts, :file_store)
+    if column_exists?(:ci_job_artifacts, :file_store)
+      remove_column(:ci_job_artifacts, :file_store)
+    end
   end
 end

db/migrate/20171214144320_add_store_column_to_uploads.rb → ee/db/migrate/20171214144320_add_store_column_to_uploads.rb

@@ -6,7 +6,15 @@ class AddStoreColumnToUploads < ActiveRecord::Migration
 
   DOWNTIME = false
 
-  def change
-    add_column :uploads, :store, :integer
+  def up
+    unless column.exists?(:uploads, :store)
+      add_column(:uploads, :store, :integer)
+    end
+  end
+
+  def down
+    if column.exists?(:uploads, :store)
+      remove_column(:uploads, :store)
+    end
   end
 end