Commit 70e0f759 authored by Valery Sizov's avatar Valery Sizov

Geo: Remove released feature flag `geo_package_file_verification`

Removes the feature flag, it was enabled by default for a long time
parent 9eff56cb
---
title: 'Geo: Remove released feature flag `geo_package_file_verification`'
merge_request: 61568
author:
type: other
...@@ -11,7 +11,7 @@ module Geo ...@@ -11,7 +11,7 @@ module Geo
override :verification_feature_flag_enabled? override :verification_feature_flag_enabled?
def self.verification_feature_flag_enabled? def self.verification_feature_flag_enabled?
Feature.enabled?(:geo_package_file_verification, default_enabled: :yaml) true
end end
def carrierwave_uploader def carrierwave_uploader
......
---
name: geo_package_file_verification
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/46998
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/271680
milestone: '13.6'
type: development
group: group::geo
default_enabled: true
...@@ -384,16 +384,6 @@ RSpec.describe Gitlab::Geo, :geo, :request_store do ...@@ -384,16 +384,6 @@ RSpec.describe Gitlab::Geo, :geo, :request_store do
expect(described_class.verification_enabled_replicator_classes).not_to include(Geo::PackageFileReplicator) expect(described_class.verification_enabled_replicator_classes).not_to include(Geo::PackageFileReplicator)
end end
end end
context 'when verification is disabled' do
before do
stub_feature_flags(geo_package_file_verification: false)
end
it 'does not return the replicator class' do
expect(described_class.verification_enabled_replicator_classes).not_to include(Geo::PackageFileReplicator)
end
end
end end
describe '.verification_max_capacity_per_replicator_class' do describe '.verification_max_capacity_per_replicator_class' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment