summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorJan Provaznik <jprovaznik@gitlab.com>2019-06-05 12:02:52 +0000
committerJan Provaznik <jprovaznik@gitlab.com>2019-06-05 12:02:52 +0000
commitd994406192ecb00e6c3b470e9ce99a2ba537cea0 (patch)
tree2ee171e4c1fea50e8e03fd1f18d6716356670a49 /app
parent2f311b532b163d372c4d39e81d2d0a85500de393 (diff)
parent977ba4cc150092107944f3a46734b2540d321dcf (diff)
downloadgitlab-ce-d994406192ecb00e6c3b470e9ce99a2ba537cea0.tar.gz
Merge branch '11925-geo-sidekiq-nodes-try-to-run-jobs-even-thought-db-is-readonly' into 'master'
Ensure DB is writable before continuing jobs See merge request gitlab-org/gitlab-ce!29134
Diffstat (limited to 'app')
-rw-r--r--app/workers/pages_domain_verification_cron_worker.rb2
-rw-r--r--app/workers/pages_domain_verification_worker.rb2
2 files changed, 4 insertions, 0 deletions
diff --git a/app/workers/pages_domain_verification_cron_worker.rb b/app/workers/pages_domain_verification_cron_worker.rb
index 92d62a15aee..60703c83e9e 100644
--- a/app/workers/pages_domain_verification_cron_worker.rb
+++ b/app/workers/pages_domain_verification_cron_worker.rb
@@ -5,6 +5,8 @@ class PagesDomainVerificationCronWorker
include CronjobQueue
def perform
+ return if Gitlab::Database.read_only?
+
PagesDomain.needs_verification.find_each do |domain|
PagesDomainVerificationWorker.perform_async(domain.id)
end
diff --git a/app/workers/pages_domain_verification_worker.rb b/app/workers/pages_domain_verification_worker.rb
index b3319ff5a13..7817b2ee5fc 100644
--- a/app/workers/pages_domain_verification_worker.rb
+++ b/app/workers/pages_domain_verification_worker.rb
@@ -5,6 +5,8 @@ class PagesDomainVerificationWorker
# rubocop: disable CodeReuse/ActiveRecord
def perform(domain_id)
+ return if Gitlab::Database.read_only?
+
domain = PagesDomain.find_by(id: domain_id)
return unless domain