Remove feature flag from BackgroundMigrationWorker
The feature has been running on GitLab.com for a while now, without any problems. This commit removes the eature flag, and enables the feature for all users.
This commit is contained in:
parent
ad985d8d6b
commit
e951a15cdc
|
@ -10,17 +10,7 @@ class BackgroundMigrationWorker
|
||||||
# maintenance related tasks have plenty of time to clean up after a migration
|
# maintenance related tasks have plenty of time to clean up after a migration
|
||||||
# has been performed.
|
# has been performed.
|
||||||
def self.minimum_interval
|
def self.minimum_interval
|
||||||
if enable_health_check?
|
2.minutes.to_i
|
||||||
2.minutes.to_i
|
|
||||||
else
|
|
||||||
5.minutes.to_i
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def self.enable_health_check?
|
|
||||||
Rails.env.development? ||
|
|
||||||
Rails.env.test? ||
|
|
||||||
Feature.enabled?('background_migration_health_check')
|
|
||||||
end
|
end
|
||||||
|
|
||||||
# Performs the background migration.
|
# Performs the background migration.
|
||||||
|
@ -86,8 +76,6 @@ class BackgroundMigrationWorker
|
||||||
# class_name - The name of the background migration that we might want to
|
# class_name - The name of the background migration that we might want to
|
||||||
# run.
|
# run.
|
||||||
def healthy_database?
|
def healthy_database?
|
||||||
return true unless self.class.enable_health_check?
|
|
||||||
|
|
||||||
return true unless Gitlab::Database.postgresql?
|
return true unless Gitlab::Database.postgresql?
|
||||||
|
|
||||||
!Postgresql::ReplicationSlot.lag_too_great?
|
!Postgresql::ReplicationSlot.lag_too_great?
|
||||||
|
|
|
@ -0,0 +1,5 @@
|
||||||
|
---
|
||||||
|
title: Remove health check feature flag in BackgroundMigrationWorker
|
||||||
|
merge_request:
|
||||||
|
author:
|
||||||
|
type: changed
|
Loading…
Reference in New Issue