Skip to content
Snippets Groups Projects
Commit 6411cedb authored by João Alexandre Cunha's avatar João Alexandre Cunha Committed by Rémy Coutable
Browse files

Removes reactive_caching_limit_environment FF

FF was already enabled by default and is working well on .com
parent 0938bff6
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -424,11 +424,6 @@ def has_metrics_and_can_query?
def generate_slug
self.slug = Gitlab::Slug::Environment.new(name).generate
end
# Overrides ReactiveCaching default to activate limit checking behind a FF
def reactive_cache_limit_enabled?
Feature.enabled?(:reactive_caching_limit_environment, project, default_enabled: true)
end
end
 
Environment.prepend_if_ee('EE::Environment')
---
title: Remove Feature Flag that controls data limit on Deploy Boards, thus making
10MB limits mandatory
merge_request: 47950
author:
type: changed
---
name: reactive_caching_limit_environment
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/34202
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/202633
milestone: '13.2'
type: development
group: group::configure
default_enabled: true
Loading
Loading
@@ -877,16 +877,6 @@
expect(described_class.reactive_cache_hard_limit).to eq(10.megabyte)
end
 
it 'overrides reactive_cache_limit_enabled? with a FF' do
environment_with_enabled_ff = build(:environment, project: create(:project))
environment_with_disabled_ff = build(:environment, project: create(:project))
stub_feature_flags(reactive_caching_limit_environment: environment_with_enabled_ff.project)
expect(environment_with_enabled_ff.send(:reactive_cache_limit_enabled?)).to be_truthy
expect(environment_with_disabled_ff.send(:reactive_cache_limit_enabled?)).to be_falsey
end
it 'returns cache data from the deployment platform' do
expect(environment.deployment_platform).to receive(:calculate_reactive_cache_for)
.with(environment).and_return(pods: %w(pod1 pod2))
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment