From 1cba51150c772c0a6e07b471db6056a91f875c15 Mon Sep 17 00:00:00 2001
From: Max Fan <mfan@gitlab.com>
Date: Wed, 7 Feb 2024 11:25:14 -0800
Subject: [PATCH] Removing FF track_ci_build_created_internal_event
Creating a CI::Build will now send an internal event
This is more performant than using database level service pings
Changelog: performance
---
app/models/ci/build.rb | 2 --
.../track_ci_build_created_internal_event.yml | 9 ---------
spec/models/ci/build_spec.rb | 12 ------------
3 files changed, 23 deletions(-)
delete mode 100644 config/feature_flags/gitlab_com_derisk/track_ci_build_created_internal_event.yml
diff --git a/app/models/ci/build.rb b/app/models/ci/build.rb
index 17613240c4faa..210352e46183b 100644
--- a/app/models/ci/build.rb
+++ b/app/models/ci/build.rb
@@ -1246,8 +1246,6 @@ def track_ci_secrets_management_id_tokens_usage
end
def track_ci_build_created_event
- return unless Feature.enabled?(:track_ci_build_created_internal_event, project, type: :gitlab_com_derisk)
-
Gitlab::InternalEvents.track_event('create_ci_build', project: project, user: user)
end
diff --git a/config/feature_flags/gitlab_com_derisk/track_ci_build_created_internal_event.yml b/config/feature_flags/gitlab_com_derisk/track_ci_build_created_internal_event.yml
deleted file mode 100644
index 2459e8b05e619..0000000000000
--- a/config/feature_flags/gitlab_com_derisk/track_ci_build_created_internal_event.yml
+++ /dev/null
@@ -1,9 +0,0 @@
----
-name: track_ci_build_created_internal_event
-feature_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/429063
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141421
-rollout_issue_url: https://gitlab.com/gitlab-com/gl-infra/production/-/issues/17409
-milestone: '16.9'
-group: group::pipeline execution
-type: gitlab_com_derisk
-default_enabled: false
diff --git a/spec/models/ci/build_spec.rb b/spec/models/ci/build_spec.rb
index 84369b9bfc337..891eb597b0dfd 100644
--- a/spec/models/ci/build_spec.rb
+++ b/spec/models/ci/build_spec.rb
@@ -102,18 +102,6 @@
build.save!
end
-
- context 'with FF track_ci_build_created_internal_event disabled' do
- before do
- stub_feature_flags(track_ci_build_created_internal_event: false)
- end
-
- it 'does not track creation event' do
- expect(Gitlab::InternalEvents).not_to receive(:track_event)
-
- create(:ci_build)
- end
- end
end
end
--
GitLab