From 1a5cc87d341dbd1bf7740f48cdb458de1f8d1fc6 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Thu, 18 Feb 2021 15:30:35 +0000 Subject: Add latest changes from gitlab-org/gitlab@13-9-stable-ee --- lib/gitlab.rb | 10 ++++++++++ spec/lib/gitlab_spec.rb | 9 +++++++-- 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 -- cgit v1.2.3