diff --git a/config/events/1655726589_ide_edit_g_edit_by_web_ide.yml b/config/events/1655726589_ide_edit_g_edit_by_web_ide.yml
deleted file mode 100644
index 128bfaf6029ab5dcc18bf735f49c9d0fd03497b4..0000000000000000000000000000000000000000
--- a/config/events/1655726589_ide_edit_g_edit_by_web_ide.yml
+++ /dev/null
@@ -1,22 +0,0 @@
----
-description: Triggered from backend on editing file in web ide
-category: ide_edit
-action: g_edit_by_web_ide
-identifiers:
-- project
-- user
-- namespace
-product_section: dev
-product_stage: create
-product_group: group::editor
-product_category: web_ide
-milestone: "15.1"
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/90484
-distributions:
-- ce
-- ee
-tiers:
-- free
-- premium
-- ultimate
-
diff --git a/config/events/1655726650_ide_edit_g_edit_by_sfe.yml b/config/events/1655726650_ide_edit_g_edit_by_sfe.yml
deleted file mode 100644
index 9acdc317cf5e79474dd8853e154898d0b8e5ba6d..0000000000000000000000000000000000000000
--- a/config/events/1655726650_ide_edit_g_edit_by_sfe.yml
+++ /dev/null
@@ -1,22 +0,0 @@
----
-description: Triggered from backend on editing file by sfe
-category: ide_edit
-action: g_edit_by_sfe
-identifiers:
- - project
- - user
- - namespace
-product_section: dev
-product_stage: create
-product_group: group::editor
-product_category: web_ide
-milestone: "15.1"
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/90484
-distributions:
- - ce
- - ee
-tiers:
- - free
- - premium
- - ultimate
-
diff --git a/config/events/1655726683_ide_edit_g_edit_by_snippet_ide.yml b/config/events/1655726683_ide_edit_g_edit_by_snippet_ide.yml
deleted file mode 100644
index 2b3ed1a5d7afe7e39a2ce2f1032c1171cbf2af77..0000000000000000000000000000000000000000
--- a/config/events/1655726683_ide_edit_g_edit_by_snippet_ide.yml
+++ /dev/null
@@ -1,22 +0,0 @@
----
-description: Triggered from backend on editing file by ide snippet
-category: ide_edit
-action: g_edit_by_snippet_ide
-identifiers:
- - project
- - user
- - namespace
-product_section: dev
-product_stage: create
-product_group: group::editor
-product_category: web_ide
-milestone: "15.1"
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/90484
-distributions:
- - ce
- - ee
-tiers:
- - free
- - premium
- - ultimate
-
diff --git a/config/events/1655726622_ide_edit_g_edit_by_live_preview.yml b/config/events/1669597397_Gitlab__UsageDataCounters__EditorUniqueCounter_ide_edit.yml
similarity index 62%
rename from config/events/1655726622_ide_edit_g_edit_by_live_preview.yml
rename to config/events/1669597397_Gitlab__UsageDataCounters__EditorUniqueCounter_ide_edit.yml
index d9d54e1c3111f115b4deafef113ac6b259d53240..67240ae2363d707b5a99e9f361a567acf2b2bc2d 100644
--- a/config/events/1655726622_ide_edit_g_edit_by_live_preview.yml
+++ b/config/events/1669597397_Gitlab__UsageDataCounters__EditorUniqueCounter_ide_edit.yml
@@ -1,7 +1,7 @@
---
-description: Triggered from backend on showing a file in live preview
-category: ide_edit
-action: g_edit_by_live_preview
+description: Triggered from backend on interaction with web ide
+category: Gitlab::UsageDataCounters::EditorUniqueCounter
+action: ide_edit
identifiers:
- project
- user
@@ -10,8 +10,8 @@ product_section: dev
product_stage: create
product_group: group::editor
product_category: web_ide
-milestone: "15.1"
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/90484
+milestone: "15.7"
+introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/104809
distributions:
- ce
- ee
@@ -19,4 +19,3 @@ tiers:
- free
- premium
- ultimate
-
diff --git a/lib/gitlab/usage_data_counters/editor_unique_counter.rb b/lib/gitlab/usage_data_counters/editor_unique_counter.rb
index 5ede840661a9175e05066c2d7a10af29ee36ab84..0b448f6815388299b957b9ff53d4033675cce817 100644
--- a/lib/gitlab/usage_data_counters/editor_unique_counter.rb
+++ b/lib/gitlab/usage_data_counters/editor_unique_counter.rb
@@ -45,20 +45,23 @@ def track_live_preview_edit_action(author:, time: Time.zone.now, project:)
private
- def track_unique_action(action, author, time, project = nil)
+ def track_unique_action(event_name, author, time, project = nil)
return unless author
if Feature.enabled?(:route_hll_to_snowplow_phase2)
Gitlab::Tracking.event(
+ name,
'ide_edit',
- action.to_s,
+ property: event_name.to_s,
project: project,
namespace: project&.namespace,
- user: author
+ user: author,
+ label: 'usage_activity_by_stage_monthly.create.action_monthly_active_users_ide_edit',
+ context: [Gitlab::Tracking::ServicePingContext.new(data_source: :redis_hll, event: event_name).to_context]
)
end
- Gitlab::UsageDataCounters::HLLRedisCounter.track_event(action, values: author.id, time: time)
+ Gitlab::UsageDataCounters::HLLRedisCounter.track_event(event_name, values: author.id, time: time)
end
def count_unique(actions, date_from, date_to)
diff --git a/spec/controllers/projects/service_ping_controller_spec.rb b/spec/controllers/projects/service_ping_controller_spec.rb
index 22fb18edc802b5a7c3633dfc4190572e2a880508..10d4b897564f03858d52e54031782bb4b5f9bee8 100644
--- a/spec/controllers/projects/service_ping_controller_spec.rb
+++ b/spec/controllers/projects/service_ping_controller_spec.rb
@@ -91,11 +91,14 @@
expect(response).to have_gitlab_http_status(:ok)
end
- it_behaves_like 'Snowplow event tracking' do
+ it_behaves_like 'Snowplow event tracking with RedisHLL context' do
let(:project) { create(:project) }
- let(:category) { 'ide_edit' }
- let(:action) { 'g_edit_by_live_preview' }
let(:namespace) { project.namespace }
+ let(:category) { 'Gitlab::UsageDataCounters::EditorUniqueCounter' }
+ let(:action) { 'ide_edit' }
+ let(:property) { 'g_edit_by_live_preview' }
+ let(:label) { 'usage_activity_by_stage_monthly.create.action_monthly_active_users_ide_edit' }
+ let(:context) { [Gitlab::Tracking::ServicePingContext.new(data_source: :redis_hll, event: event_name).to_context] }
let(:feature_flag_name) { :route_hll_to_snowplow_phase2 }
end
end
diff --git a/spec/requests/api/commits_spec.rb b/spec/requests/api/commits_spec.rb
index 6ef9cf70a660099c30bee453601717ecf7e438dc..3b686fb659845d79d8e18c9103157cd2f2b6ab64 100644
--- a/spec/requests/api/commits_spec.rb
+++ b/spec/requests/api/commits_spec.rb
@@ -492,10 +492,13 @@
subject
end
- it_behaves_like 'Snowplow event tracking' do
- let(:namespace) { project.namespace }
- let(:category) { 'ide_edit' }
- let(:action) { 'g_edit_by_web_ide' }
+ it_behaves_like 'Snowplow event tracking with RedisHLL context' do
+ let(:namespace) { project.namespace.reload }
+ let(:category) { 'Gitlab::UsageDataCounters::EditorUniqueCounter' }
+ let(:action) { 'ide_edit' }
+ let(:property) { 'g_edit_by_web_ide' }
+ let(:label) { 'usage_activity_by_stage_monthly.create.action_monthly_active_users_ide_edit' }
+ let(:context) { [Gitlab::Tracking::ServicePingContext.new(data_source: :redis_hll, event: event_name).to_context] }
let(:feature_flag_name) { :route_hll_to_snowplow_phase2 }
end
diff --git a/spec/requests/api/graphql/mutations/snippets/update_spec.rb b/spec/requests/api/graphql/mutations/snippets/update_spec.rb
index 1a5d3620f222db908658b27f3412ae9a3b3af1cf..a385c96de612c37107160da630dd190aafe04e08 100644
--- a/spec/requests/api/graphql/mutations/snippets/update_spec.rb
+++ b/spec/requests/api/graphql/mutations/snippets/update_spec.rb
@@ -192,11 +192,18 @@ def blob_at(filename)
stub_session('warden.user.user.key' => [[current_user.id], current_user.authenticatable_salt])
end
- it_behaves_like 'Snowplow event tracking' do
+ it_behaves_like 'Snowplow event tracking with RedisHLL context' do
+ let(:feature_flag_name) { :route_hll_to_snowplow_phase2 }
let(:user) { current_user }
+ let(:property) { 'g_edit_by_snippet_ide' }
let(:namespace) { project.namespace }
- let(:category) { 'ide_edit' }
- let(:action) { 'g_edit_by_snippet_ide' }
+ let(:category) { 'Gitlab::UsageDataCounters::EditorUniqueCounter' }
+ let(:action) { 'ide_edit' }
+ let(:label) { 'usage_activity_by_stage_monthly.create.action_monthly_active_users_ide_edit' }
+ let(:context) do
+ [Gitlab::Tracking::ServicePingContext.new(data_source: :redis_hll, event: event_name).to_context]
+ end
+
let(:feature_flag_name) { :route_hll_to_snowplow_phase2 }
end
end