Ottomata has submitted this change and it was merged.
Change subject: Move burrow role to kafka/analytics
......................................................................
Move burrow role to kafka/analytics
Note that this commit removes the last manifests/role/analytics/* file
Bug: T109859
Change-Id: Id19308159659c84d480fe5aa02356438d6c50da1
---
M manifests/site.pp
R modules/role/manifests/kafka/analytics/burrow.pp
2 files changed, 6 insertions(+), 6 deletions(-)
Approvals:
Ottomata: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/site.pp b/manifests/site.pp
index ea88a7e..c872d04 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -5,7 +5,6 @@
import 'misc/*.pp'
import 'network.pp'
import 'role/*.pp'
-import 'role/analytics/*.pp'
# Base nodes
@@ -1244,10 +1243,10 @@
# virtual machine for misc. PHP apps
node 'krypton.eqiad.wmnet' {
- # analytics::burrow is a Kafka consumer lag monitor.
+ # kafka::analytics::burrow is a Kafka consumer lag monitor.
# Running this here because krypton is a 'misc' Jessie
# <s>monitoring host</s> (not really, it's just misc apps)
- role wikimania_scholarships, iegreview, grafana, gdash, analytics::burrow,
racktables
+ role wikimania_scholarships, iegreview, grafana, gdash,
kafka::analytics::burrow, racktables
include standard
}
diff --git a/manifests/role/analytics/burrow.pp
b/modules/role/manifests/kafka/analytics/burrow.pp
similarity index 90%
rename from manifests/role/analytics/burrow.pp
rename to modules/role/manifests/kafka/analytics/burrow.pp
index 0be5255..493cc3b 100644
--- a/manifests/role/analytics/burrow.pp
+++ b/modules/role/manifests/kafka/analytics/burrow.pp
@@ -1,7 +1,8 @@
-# == role/analytics/burrow.pp
+# == Class role::kafka::analytics::burrow
# Role classes for burrow, a consumer offset lag monitoring tool
-# for Kafka
-class role::analytics::burrow {
+# for Kafka.
+#
+class role::kafka::analytics::burrow {
# Include the kafka config role to get all configuration data
include role::kafka::analytics::config
--
To view, visit https://gerrit.wikimedia.org/r/270859
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id19308159659c84d480fe5aa02356438d6c50da1
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ottomata <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
