Elukey has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/391179 )
Change subject: role::prometheus::analytics: remove redundant target configs
......................................................................
role::prometheus::analytics: remove redundant target configs
Bug: T177459
Change-Id: Ica4ee00c27c0de9a23bc32d833f649b30d782b07
---
M modules/role/manifests/prometheus/analytics.pp
1 file changed, 6 insertions(+), 24 deletions(-)
Approvals:
Elukey: Looks good to me, approved
jenkins-bot: Verified
Filippo Giunchedi: Looks good to me, but someone else must approve
diff --git a/modules/role/manifests/prometheus/analytics.pp
b/modules/role/manifests/prometheus/analytics.pp
index 3bb3447..70cfc1f 100644
--- a/modules/role/manifests/prometheus/analytics.pp
+++ b/modules/role/manifests/prometheus/analytics.pp
@@ -34,33 +34,15 @@
},
]
- prometheus::jmx_exporter_config{ "druid_broker_${::site}":
- dest => "${targets_path}/jmx_druid_broker_${::site}.yaml",
- class_name => 'profile::druid::broker',
+ prometheus::jmx_exporter_config{ "druid_public_${::site}":
+ dest => "${targets_path}/jmx_druid_public_${::site}.yaml",
+ class_name => 'role::druid::public::worker',
site => $::site,
}
- prometheus::jmx_exporter_config{ "druid_overlord_${::site}":
- dest => "${targets_path}/jmx_druid_overlord_${::site}.yaml",
- class_name => 'profile::druid::overlord',
- site => $::site,
- }
-
- prometheus::jmx_exporter_config{ "druid_middlemanager_${::site}":
- dest => "${targets_path}/jmx_druid_middlemanager_${::site}.yaml",
- class_name => 'profile::druid::middlemanager',
- site => $::site,
- }
-
- prometheus::jmx_exporter_config{ "druid_coordinator_${::site}":
- dest => "${targets_path}/jmx_druid_coordinator_${::site}.yaml",
- class_name => 'profile::druid::coordinator',
- site => $::site,
- }
-
- prometheus::jmx_exporter_config{ "druid_historical_${::site}":
- dest => "${targets_path}/jmx_druid_historical_${::site}.yaml",
- class_name => 'profile::druid::historical',
+ prometheus::jmx_exporter_config{ "druid_analytics_${::site}":
+ dest => "${targets_path}/jmx_druid_analytics_${::site}.yaml",
+ class_name => 'role::druid::analytics::worker',
site => $::site,
}
--
To view, visit https://gerrit.wikimedia.org/r/391179
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ica4ee00c27c0de9a23bc32d833f649b30d782b07
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Elukey <ltosc...@wikimedia.org>
Gerrit-Reviewer: Elukey <ltosc...@wikimedia.org>
Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@wikimedia.org>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
