Merge branch '321786-update-usage-ping-metrics-definitions-for-group-apm' into 'master'

Deprecate projects_with_prometheus_alerts and pod_logs_usages_total metrics

See merge request gitlab-org/gitlab!61872

config/metrics/counts_28d/20210216180955_projects_with_prometheus_alerts.yml

@@ -6,7 +6,7 @@ product_stage:
 product_group: group::monitor
 product_category: metrics
 value_type: number
-status: data_available
+status: deprecated
 time_frame: 28d
 data_source: database
 distribution:

config/metrics/counts_all/20210216175019_projects_with_prometheus_alerts.yml

@@ -3,10 +3,10 @@ key_path: counts.projects_with_prometheus_alerts
 description: Projects with Prometheus alerting enabled
 product_section: ops
 product_stage: monitor
-product_group: group::apm
+product_group: group::monitor
 product_category: metrics
 value_type: number
-status: data_available
+status: deprecated
 time_frame: all
 data_source: database
 distribution:

config/metrics/counts_all/20210216175021_pod_logs_usages_total.yml

@@ -3,10 +3,10 @@ key_path: counts.pod_logs_usages_total
 description: Count the total number of log views
 product_section: ops
 product_stage: monitor
-product_group: group::apm
+product_group: group::monitor
 product_category: logging
 value_type: number
-status: data_available
+status: deprecated
 time_frame: all
 data_source: database
 distribution:

config/metrics/counts_all/20210216175403_projects_with_prometheus_alerts.yml

@@ -6,7 +6,7 @@ product_stage:
 product_group: group::configure
 product_category: kubernetes_management
 value_type: number
-status: data_available
+status: deprecated
 time_frame: all
 data_source:
 distribution:

doc/development/usage_ping/dictionary.md

@@ -4120,9 +4120,9 @@ Count the total number of log views
 
 [YAML definition](https://gitlab.com/gitlab-org/gitlab/-/blob/master/config/metrics/counts_all/20210216175021_pod_logs_usages_total.yml)
 
-Group: `group::apm`
+Group: `group::monitor`
 
-Status: `data_available`
+Status: `deprecated`
 
 Tiers: `free`
 
@@ -5500,9 +5500,9 @@ Projects with Prometheus alerting enabled
 
 [YAML definition](https://gitlab.com/gitlab-org/gitlab/-/blob/master/config/metrics/counts_all/20210216175019_projects_with_prometheus_alerts.yml)
 
-Group: `group::apm`
+Group: `group::monitor`
 
-Status: `data_available`
+Status: `deprecated`
 
 Tiers: `free`
 
@@ -15896,7 +15896,7 @@ Projects with Prometheus alerting enabled
 
 Group: `group::configure`
 
-Status: `data_available`
+Status: `deprecated`
 
 Tiers: `free`, `premium`, `ultimate`
 
@@ -17758,7 +17758,7 @@ Projects with Prometheus alerting enabled
 
 Group: `group::monitor`
 
-Status: `data_available`
+Status: `deprecated`
 
 Tiers: `free`, `premium`, `ultimate`
 

ee/lib/ee/gitlab/usage_data.rb

@@ -201,7 +201,7 @@ module EE
                 issues_with_health_status: count(::Issue.with_health_status, start: minimum_id(::Issue), finish: maximum_id(::Issue)),
                 ldap_keys: count(::LDAPKey),
                 ldap_users: count(::User.ldap, 'users.id'),
-                pod_logs_usages_total: redis_usage_data { ::Gitlab::UsageCounters::PodLogs.usage_totals[:total] },
+                pod_logs_usages_total: ::Gitlab::UsageData::DEPRECATED_VALUE,
                 merged_merge_requests_using_approval_rules: count(::MergeRequest.merged.joins(:approval_rules), # rubocop: disable CodeReuse/ActiveRecord
                                                                   start: minimum_id(::MergeRequest),
                                                                   finish: maximum_id(::MergeRequest)),
@@ -234,7 +234,7 @@ module EE
           super.merge({
             projects_slack_notifications_active: distinct_count(::Project.with_slack_service.where(time_period), :creator_id),
             projects_slack_slash_active: distinct_count(::Project.with_slack_slash_commands_service.where(time_period), :creator_id),
-            projects_with_prometheus_alerts: distinct_count(::Project.with_prometheus_service.where(time_period), :creator_id)
+            projects_with_prometheus_alerts: ::Gitlab::UsageData::DEPRECATED_VALUE
           })
         end
 

ee/spec/lib/ee/gitlab/usage_data_spec.rb

@@ -125,7 +125,7 @@ RSpec.describe Gitlab::UsageData do
         network_policy_drops
       ))
 
-      expect(count_data[:projects_with_prometheus_alerts]).to eq(2)
+      expect(count_data[:projects_with_prometheus_alerts]).to eq(Gitlab::UsageData::DEPRECATED_VALUE)
       expect(count_data[:feature_flags]).to eq(1)
       expect(count_data[:status_page_projects]).to eq(1)
       expect(count_data[:status_page_issues]).to eq(1)
@@ -268,12 +268,12 @@ RSpec.describe Gitlab::UsageData do
       expect(described_class.usage_activity_by_stage_configure({})).to include(
         projects_slack_notifications_active: 2,
         projects_slack_slash_active: 2,
-        projects_with_prometheus_alerts: 2
+        projects_with_prometheus_alerts: Gitlab::UsageData::DEPRECATED_VALUE
       )
       expect(described_class.usage_activity_by_stage_configure(described_class.monthly_time_range_db_params)).to include(
         projects_slack_notifications_active: 1,
         projects_slack_slash_active: 1,
-        projects_with_prometheus_alerts: 1
+        projects_with_prometheus_alerts: Gitlab::UsageData::DEPRECATED_VALUE
       )
     end
   end

lib/gitlab/usage_data.rb

@@ -166,7 +166,7 @@ module Gitlab
             projects_with_error_tracking_enabled: count(::ErrorTracking::ProjectErrorTrackingSetting.where(enabled: true)),
             projects_with_alerts_created: distinct_count(::AlertManagement::Alert, :project_id),
             projects_with_enabled_alert_integrations: distinct_count(::AlertManagement::HttpIntegration.active, :project_id),
-            projects_with_prometheus_alerts: distinct_count(PrometheusAlert, :project_id),
+            projects_with_prometheus_alerts: DEPRECATED_VALUE,
             projects_with_terraform_reports: distinct_count(::Ci::JobArtifact.terraform_reports, :project_id),
             projects_with_terraform_states: distinct_count(::Terraform::State, :project_id),
             protected_branches: count(ProtectedBranch),

spec/lib/gitlab/usage_data_spec.rb

@@ -576,7 +576,7 @@ RSpec.describe Gitlab::UsageData, :aggregate_failures do
       expect(count_data[:projects_with_error_tracking_enabled]).to eq(1)
       expect(count_data[:projects_with_tracing_enabled]).to eq(1)
       expect(count_data[:projects_with_enabled_alert_integrations]).to eq(1)
-      expect(count_data[:projects_with_prometheus_alerts]).to eq(2)
+      expect(count_data[:projects_with_prometheus_alerts]).to eq(Gitlab::UsageData::DEPRECATED_VALUE)
       expect(count_data[:projects_with_terraform_reports]).to eq(2)
       expect(count_data[:projects_with_terraform_states]).to eq(2)
       expect(count_data[:projects_with_alerts_created]).to eq(1)