Skip to content
Snippets Groups Projects
Commit 6baa402f authored by Albert Salim's avatar Albert Salim
Browse files

Fixme: Remove Gitlab.com? check for DB pipeline

parent 90413d66
No related merge requests found
......@@ -13,14 +13,14 @@ class AddTmpIndexJobArtifactsIdAndExpireAt < Gitlab::Database::Migration[2.0]
SQL
def up
return if Gitlab.com?
# return if Gitlab.com?
add_concurrent_index :ci_job_artifacts, [:id, :expire_at, :file_type],
where: EXPIRE_AT_ON_22_MIDNIGHT_IN_TIMEZONE_OR_TRACE, name: INDEX_NAME
end
def down
return if Gitlab.com?
# return if Gitlab.com?
remove_concurrent_index_by_name :ci_job_artifacts, INDEX_NAME
end
......
......@@ -3,7 +3,9 @@
class FixIncorrectJobArtifactsExpireAt < Gitlab::Database::Migration[2.0]
disable_ddl_transaction!
restrict_gitlab_migration gitlab_schema: :gitlab_ci
# FIXME: Revert to :gitlab_ci.
# :gitlab_main is required during review because db test pipeline is not configured to use ci
restrict_gitlab_migration gitlab_schema: :gitlab_main
MIGRATION = 'RemoveBackfilledJobArtifactsExpireAt'
......@@ -11,7 +13,7 @@ class FixIncorrectJobArtifactsExpireAt < Gitlab::Database::Migration[2.0]
INTERVAL = 2.minutes.freeze
def up
return if Gitlab.com?
# return if Gitlab.com?
queue_batched_background_migration(
MIGRATION,
......@@ -23,7 +25,7 @@ def up
end
def down
return if Gitlab.com?
# return if Gitlab.com?
delete_batched_background_migration(MIGRATION, :ci_job_artifacts, :id, [])
end
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment