Commit 43fea56d authored by Sean Arnold's avatar Sean Arnold

Deprecate monthly incident_sla_enabled metric

parent 39393b4e
...@@ -4354,6 +4354,8 @@ The total count of Helm packages that have been published. ...@@ -4354,6 +4354,8 @@ The total count of Helm packages that have been published.
Group: `group::package` Group: `group::package`
Data Category: `Optional`
Status: `implemented` Status: `implemented`
Tiers: `free`, `premium`, `ultimate` Tiers: `free`, `premium`, `ultimate`
...@@ -22264,7 +22266,7 @@ Group: `group::monitor` ...@@ -22264,7 +22266,7 @@ Group: `group::monitor`
Data Category: `Optional` Data Category: `Optional`
Status: `data_available` Status: `deprecated`
Tiers: `premium`, `ultimate` Tiers: `premium`, `ultimate`
......
...@@ -7,7 +7,7 @@ product_stage: ...@@ -7,7 +7,7 @@ product_stage:
product_group: group::monitor product_group: group::monitor
product_category: metrics product_category: metrics
value_type: number value_type: number
status: data_available status: deprecated
time_frame: all time_frame: all
data_source: database data_source: database
distribution: distribution:
......
...@@ -320,7 +320,7 @@ module EE ...@@ -320,7 +320,7 @@ module EE
def usage_activity_by_stage_monitor(time_period) def usage_activity_by_stage_monitor(time_period)
super.merge({ super.merge({
operations_dashboard_users_with_projects_added: distinct_count(UsersOpsDashboardProject.joins(:user).merge(::User.active).where(time_period), :user_id), operations_dashboard_users_with_projects_added: distinct_count(UsersOpsDashboardProject.joins(:user).merge(::User.active).where(time_period), :user_id),
projects_incident_sla_enabled: count(::Project.with_enabled_incident_sla) projects_incident_sla_enabled: time_period.present? ? ::Gitlab::UsageData::DEPRECATED_VALUE : count(::Project.with_enabled_incident_sla)
}) })
end end
......
...@@ -477,8 +477,7 @@ RSpec.describe Gitlab::UsageData do ...@@ -477,8 +477,7 @@ RSpec.describe Gitlab::UsageData do
projects_incident_sla_enabled: 2 projects_incident_sla_enabled: 2
) )
expect(described_class.usage_activity_by_stage_monitor(described_class.monthly_time_range_db_params)).to include( expect(described_class.usage_activity_by_stage_monitor(described_class.monthly_time_range_db_params)).to include(
operations_dashboard_users_with_projects_added: 1, operations_dashboard_users_with_projects_added: 1
projects_incident_sla_enabled: 2
) )
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