Commit 0c176714 authored by Robert Speicher's avatar Robert Speicher

Merge branch...

Merge branch '322166-remove-usage_data_o_pipeline_authoring_unique_users_pushing_mr_ciconfigfile' into 'master'

Remove the FF *_unique_users_pushing_mr_ciconfigfile [RUN ALL RSPEC] [RUN AS-IF-FOSS]

See merge request gitlab-org/gitlab!62283
parents 85744649 35e5175f
---
name: usage_data_o_pipeline_authoring_unique_users_pushing_mr_ciconfigfile
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/54707
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/322166
milestone: '13.10'
type: development
group: group::pipeline authoring
default_enabled: true
...@@ -355,7 +355,6 @@ ...@@ -355,7 +355,6 @@
category: pipeline_authoring category: pipeline_authoring
redis_slot: pipeline_authoring redis_slot: pipeline_authoring
aggregation: weekly aggregation: weekly
feature_flag: usage_data_o_pipeline_authoring_unique_users_pushing_mr_ciconfigfile
# Merge request widgets # Merge request widgets
- name: users_expanding_secure_security_report - name: users_expanding_secure_security_report
redis_slot: secure redis_slot: secure
......
...@@ -189,7 +189,6 @@ module Gitlab ...@@ -189,7 +189,6 @@ module Gitlab
end end
def track_mr_including_ci_config(user:, merge_request:) def track_mr_including_ci_config(user:, merge_request:)
return unless Feature.enabled?(:usage_data_o_pipeline_authoring_unique_users_pushing_mr_ciconfigfile, user, default_enabled: :yaml)
return unless merge_request.includes_ci_config? return unless merge_request.includes_ci_config?
track_unique_action_by_user(MR_INCLUDING_CI_CONFIG_ACTION, user) track_unique_action_by_user(MR_INCLUDING_CI_CONFIG_ACTION, user)
......
...@@ -352,14 +352,6 @@ RSpec.describe Gitlab::UsageDataCounters::MergeRequestActivityUniqueCounter, :cl ...@@ -352,14 +352,6 @@ RSpec.describe Gitlab::UsageDataCounters::MergeRequestActivityUniqueCounter, :cl
it_behaves_like 'a tracked merge request unique event' do it_behaves_like 'a tracked merge request unique event' do
let(:action) { described_class::MR_INCLUDING_CI_CONFIG_ACTION } let(:action) { described_class::MR_INCLUDING_CI_CONFIG_ACTION }
end end
context 'when FF usage_data_o_pipeline_authoring_unique_users_pushing_mr_ciconfigfile is disabled' do
before do
stub_feature_flags(usage_data_o_pipeline_authoring_unique_users_pushing_mr_ciconfigfile: false)
end
it_behaves_like 'not tracked merge request unique event'
end
end end
context 'when merge request does not include any ci config change' do context 'when merge request does not include any ci config change' 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