Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2021-02-18 18:30:35 +0300
committerGitLab Bot <gitlab-bot@gitlab.com>2021-02-18 18:30:35 +0300
commit1a5cc87d341dbd1bf7740f48cdb458de1f8d1fc6 (patch)
treea11f3682b329c2c1f5c37fb201134ba2c254e65f
parenta0a166e723005aea66e278c653542eb4e5cca11f (diff)
Add latest changes from gitlab-org/gitlab@13-9-stable-ee
-rw-r--r--lib/gitlab.rb10
-rw-r--r--spec/lib/gitlab_spec.rb9
2 files changed, 17 insertions, 2 deletions
diff --git a/lib/gitlab.rb b/lib/gitlab.rb
index f943b532454..ebf2cdd875a 100644
--- a/lib/gitlab.rb
+++ b/lib/gitlab.rb
@@ -123,6 +123,16 @@ module Gitlab
def self.maintenance_mode?
return false unless ::Gitlab::CurrentSettings.current_application_settings?
+ # `maintenance_mode` column was added to the `current_settings` table in 13.2
+ # When upgrading from < 13.2 to >=13.8 `maintenance_mode` will not be
+ # found in settings.
+ # `Gitlab::CurrentSettings#uncached_application_settings` in
+ # lib/gitlab/current_settings.rb is expected to handle such cases, and use
+ # the default value for the setting instead, but in this case, it doesn't,
+ # see https://gitlab.com/gitlab-org/gitlab/-/issues/321836
+ # As a work around, we check if the setting method is available
+ return false unless ::Gitlab::CurrentSettings.respond_to?(:maintenance_mode)
+
::Gitlab::CurrentSettings.maintenance_mode
end
end
diff --git a/spec/lib/gitlab_spec.rb b/spec/lib/gitlab_spec.rb
index 5f945d5b9fc..c5738ae730f 100644
--- a/spec/lib/gitlab_spec.rb
+++ b/spec/lib/gitlab_spec.rb
@@ -363,8 +363,13 @@ RSpec.describe Gitlab do
expect(described_class.maintenance_mode?).to eq(false)
end
- it 'returns false when maintenance mode feature flag is disabled' do
- stub_feature_flags(maintenance_mode: false)
+ it 'returns false when maintenance mode column is not present' do
+ stub_maintenance_mode_setting(true)
+
+ allow(::Gitlab::CurrentSettings.current_application_settings)
+ .to receive(:respond_to?)
+ .with(:maintenance_mode, false)
+ .and_return(false)
expect(described_class.maintenance_mode?).to eq(false)
end