summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2021-07-03 12:08:08 +0000
committerGitLab Bot <gitlab-bot@gitlab.com>2021-07-03 12:08:08 +0000
commit0c922a0a151a3904fb0f18572be39c904b5c1ab6 (patch)
tree84c0aa318c84eeb15437068f8c9bd5a14dc67106
parent840b3aceff471404e2deb1c813c1ac94508db6c1 (diff)
downloadgitlab-ce-0c922a0a151a3904fb0f18572be39c904b5c1ab6.tar.gz
Add latest changes from gitlab-org/gitlab@master
-rw-r--r--config/feature_flags/development/use_workhorse_s3_client.yml8
-rw-r--r--doc/administration/object_storage.md15
-rw-r--r--lib/object_storage/direct_upload.rb1
-rw-r--r--spec/lib/object_storage/direct_upload_spec.rb10
4 files changed, 0 insertions, 34 deletions
diff --git a/config/feature_flags/development/use_workhorse_s3_client.yml b/config/feature_flags/development/use_workhorse_s3_client.yml
deleted file mode 100644
index 3c34c420ebf..00000000000
--- a/config/feature_flags/development/use_workhorse_s3_client.yml
+++ /dev/null
@@ -1,8 +0,0 @@
----
-name: use_workhorse_s3_client
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/35480
-rollout_issue_url:
-milestone: '13.2'
-type: development
-group: group::source code
-default_enabled: true
diff --git a/doc/administration/object_storage.md b/doc/administration/object_storage.md
index c2c32f059c9..525b41359cf 100644
--- a/doc/administration/object_storage.md
+++ b/doc/administration/object_storage.md
@@ -718,21 +718,6 @@ must be fulfilled:
[ETag mismatch errors](#etag-mismatch) occur if server side
encryption headers are used without enabling the Workhorse S3 client.
-##### Disabling the feature
-
-The Workhorse S3 client is enabled by default when the
-[`use_iam_profile` configuration option](#iam-permissions) is set to `true` or consolidated
-object storage settings are configured.
-
-The feature can be disabled using the `:use_workhorse_s3_client` feature flag. To disable the
-feature, ask a GitLab administrator with
-[Rails console access](feature_flags.md#how-to-enable-and-disable-features-behind-flags) to run the
-following command:
-
-```ruby
-Feature.disable(:use_workhorse_s3_client)
-```
-
#### IAM Permissions
To set up an instance profile:
diff --git a/lib/object_storage/direct_upload.rb b/lib/object_storage/direct_upload.rb
index 7fbf01f3768..3dd4e5e27d4 100644
--- a/lib/object_storage/direct_upload.rb
+++ b/lib/object_storage/direct_upload.rb
@@ -112,7 +112,6 @@ module ObjectStorage
end
def use_workhorse_s3_client?
- return false unless Feature.enabled?(:use_workhorse_s3_client, default_enabled: true)
return false unless config.use_iam_profile? || config.consolidated_settings?
# The Golang AWS SDK does not support V2 signatures
return false unless credentials.fetch(:aws_signature_version, 4).to_i >= 4
diff --git a/spec/lib/object_storage/direct_upload_spec.rb b/spec/lib/object_storage/direct_upload_spec.rb
index 4b374452c0a..006f4f603b6 100644
--- a/spec/lib/object_storage/direct_upload_spec.rb
+++ b/spec/lib/object_storage/direct_upload_spec.rb
@@ -136,16 +136,6 @@ RSpec.describe ObjectStorage::DirectUpload do
end
end
- context 'when feature flag is disabled' do
- before do
- stub_feature_flags(use_workhorse_s3_client: false)
- end
-
- it 'does not enable Workhorse client' do
- expect(subject[:UseWorkhorseClient]).to be false
- end
- end
-
context 'when V2 signatures are used' do
before do
credentials[:aws_signature_version] = 2