Commit d4e7fb04 authored by Maxime Orefice's avatar Maxime Orefice

Remove ci_decompose_for_namespace_monthly_usage_query feature flag

Changelog: other
parent 18d6c174
---
name: ci_decompose_for_namespace_monthly_usage_query
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/77952
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/350146
milestone: '14.7'
type: development
group: group::pipeline execution
default_enabled: false
...@@ -12,17 +12,10 @@ module Ci ...@@ -12,17 +12,10 @@ module Ci
scope :current_month, -> { where(date: beginning_of_month) } scope :current_month, -> { where(date: beginning_of_month) }
scope :for_namespace_monthly_usage, -> (namespace_monthly_usage) do scope :for_namespace_monthly_usage, -> (namespace_monthly_usage) do
if Feature.enabled?(:ci_decompose_for_namespace_monthly_usage_query, namespace_monthly_usage.namespace, default_enabled: :yaml)
where( where(
date: namespace_monthly_usage.date, date: namespace_monthly_usage.date,
project: Ci::ProjectMirror.by_namespace_id(namespace_monthly_usage.namespace_id).select(:project_id) project: Ci::ProjectMirror.by_namespace_id(namespace_monthly_usage.namespace_id).select(:project_id)
) )
else
where(
date: namespace_monthly_usage.date,
project: namespace_monthly_usage.namespace.projects
).allow_cross_joins_across_databases(url: 'https://gitlab.com/gitlab-org/gitlab/-/issues/343301')
end
end end
def self.beginning_of_month(time = Time.current) def self.beginning_of_month(time = Time.current)
......
...@@ -77,7 +77,6 @@ RSpec.describe Ci::Minutes::ProjectMonthlyUsage do ...@@ -77,7 +77,6 @@ RSpec.describe Ci::Minutes::ProjectMonthlyUsage do
end end
describe '.for_namespace_monthly_usage' do describe '.for_namespace_monthly_usage' do
shared_examples 'namespace monthly usage' do
let(:date_for_usage) { Date.new(2021, 5, 1) } let(:date_for_usage) { Date.new(2021, 5, 1) }
let(:namespace_usage) { create(:ci_namespace_monthly_usage, namespace: project.namespace, amount_used: 50, date: date_for_usage) } let(:namespace_usage) { create(:ci_namespace_monthly_usage, namespace: project.namespace, amount_used: 50, date: date_for_usage) }
...@@ -98,17 +97,4 @@ RSpec.describe Ci::Minutes::ProjectMonthlyUsage do ...@@ -98,17 +97,4 @@ RSpec.describe Ci::Minutes::ProjectMonthlyUsage do
end end
end end
end end
context 'when ci_decompose_for_namespace_monthly_usage_query is enabled' do
it_behaves_like 'namespace monthly usage'
end
context 'when ci_decompose_for_namespace_monthly_usage_query is disabled' do
before do
stub_feature_flags(ci_decompose_for_namespace_monthly_usage_query: false)
end
it_behaves_like 'namespace monthly usage'
end
end
end end
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