Add latest changes from gitlab-org/gitlab@master

1 files changed, 5 insertions, 16 deletions

diff --git a/config/object_store_settings.rb b/config/object_store_settings.rb
index 5603ea51672..f92bffd45db 100644
--- a/config/object_store_settings.rb
+++ b/config/object_store_settings.rb
@@ -124,9 +124,12 @@ class ObjectStoreSettings
       target_config = common_config.merge(overrides.slice(*ALLOWED_OBJECT_STORE_OVERRIDES))
       section = settings.try(store_type)
 
-      next unless uses_object_storage?(section)
+      # Admins can selectively disable object storage for a specific
+      # type as an override in the consolidated settings.
+      next unless overrides.fetch('enabled', true)
+      next unless section
 
-      if target_config['bucket'].blank?
+      if section['enabled'] && target_config['bucket'].blank?
         missing_bucket_for(store_type)
         next
       end
@@ -146,20 +149,6 @@ class ObjectStoreSettings
 
   private
 
-  # Admins can selectively disable object storage for a specific type. If
-  # this hasn't been set, we assume that the consolidated settings
-  # should be used.
-  def uses_object_storage?(section)
-    # Use to_h to avoid https://gitlab.com/gitlab-org/gitlab/-/issues/286873
-    section = section.to_h
-
-    enabled_globally = section.fetch('enabled', false)
-    object_store_settings = section.fetch('object_store', {})
-    os_enabled = object_store_settings.fetch('enabled', true)
-
-    enabled_globally && os_enabled
-  end
-
   # We only can use the common object storage settings if:
   # 1. The common settings are defined
   # 2. The legacy settings are not defined