Commit c10e6187 authored by allison.browne's avatar allison.browne Committed by Ash McKenzie

Mark CheckPrometheusHealthWorker as externally dependent

Add worker_has_external_dependencies! to health check worker. Add limit
to dev demo projects. Correct a typo.
parent b4178d43
...@@ -629,7 +629,7 @@ ...@@ -629,7 +629,7 @@
:tags: [] :tags: []
- :name: incident_management:clusters_applications_check_prometheus_health - :name: incident_management:clusters_applications_check_prometheus_health
:feature_category: :incident_management :feature_category: :incident_management
:has_external_dependencies: :has_external_dependencies: true
:urgency: :low :urgency: :low
:resource_boundary: :unknown :resource_boundary: :unknown
:weight: 2 :weight: 2
......
...@@ -14,6 +14,7 @@ module Clusters ...@@ -14,6 +14,7 @@ module Clusters
urgency :low urgency :low
idempotent! idempotent!
worker_has_external_dependencies!
def perform def perform
demo_project_ids = Gitlab::Monitor::DemoProjects.primary_keys demo_project_ids = Gitlab::Monitor::DemoProjects.primary_keys
......
...@@ -17,7 +17,7 @@ module Gitlab ...@@ -17,7 +17,7 @@ module Gitlab
elsif ::Gitlab.staging? elsif ::Gitlab.staging?
STAGING_IDS STAGING_IDS
elsif ::Gitlab.dev_or_test_env? elsif ::Gitlab.dev_or_test_env?
Project.pluck(:id) # rubocop: disable CodeReuse/ActiveRecord Project.limit(100).pluck(:id) # rubocop: disable CodeReuse/ActiveRecord
else else
[] []
end end
......
...@@ -111,7 +111,7 @@ describe Gitlab do ...@@ -111,7 +111,7 @@ describe Gitlab do
expect(subject).to eq true expect(subject).to eq true
end end
it 'is flase when not on staging' do it 'is false when not on staging' do
stub_config_setting(url: 'https://example.gitlab.com') stub_config_setting(url: 'https://example.gitlab.com')
expect(subject).to eq false expect(subject).to eq false
......
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