Muehlenhoff has submitted this change and it was merged.
Change subject: kafka brokers: Use DOMAIN_NETWORKS
......................................................................
kafka brokers: Use DOMAIN_NETWORKS
There's actually a few Kafka brokers in labs for testing, so rather use
DOMAIN_NETWORKS instead of limiting to the production networks.
Change-Id: I869884a65aa78c86aec1afa88180b112f32a7fa2
---
M modules/role/manifests/kafka/analytics/broker.pp
M modules/role/manifests/kafka/analytics/burrow.pp
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Elukey: Looks good to me, but someone else must approve
Muehlenhoff: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/role/manifests/kafka/analytics/broker.pp
b/modules/role/manifests/kafka/analytics/broker.pp
index 70e9f90..9e7e382 100644
--- a/modules/role/manifests/kafka/analytics/broker.pp
+++ b/modules/role/manifests/kafka/analytics/broker.pp
@@ -117,7 +117,7 @@
# TODO: $::confluent::kafka::broker::port doesn't
# seem to work as expected. Hardcoding this for now.
port => 9092,
- srange => '$PRODUCTION_NETWORKS',
+ srange => '$DOMAIN_NETWORKS',
}
include ::ferm::ipsec_allow
diff --git a/modules/role/manifests/kafka/analytics/burrow.pp
b/modules/role/manifests/kafka/analytics/burrow.pp
index 74c0d6f..0c348cd 100644
--- a/modules/role/manifests/kafka/analytics/burrow.pp
+++ b/modules/role/manifests/kafka/analytics/burrow.pp
@@ -22,6 +22,6 @@
ferm::service { 'burrow':
proto => 'tcp',
port => '8000',
- srange => '$PRODUCTION_NETWORKS',
+ srange => '$DOMAIN_NETWORKS',
}
}
--
To view, visit https://gerrit.wikimedia.org/r/297368
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I869884a65aa78c86aec1afa88180b112f32a7fa2
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Elukey <[email protected]>
Gerrit-Reviewer: Muehlenhoff <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
