Ottomata has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/394383 )

Change subject: Disable ssl for kafka-jumbo
......................................................................


Disable ssl for kafka-jumbo

Since ACL auth is enabled (for logging) this wasn't quite like the setup in 
labs.
I need to figure out how to grant cluster ACL permissions properly first.

Bug: T121561
Change-Id: I54a432ed9f512640ef34e6fc7adc9c29e5a4e99d
---
M hieradata/role/common/kafka/jumbo/broker.yaml
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Ottomata: Verified; Looks good to me, approved



diff --git a/hieradata/role/common/kafka/jumbo/broker.yaml 
b/hieradata/role/common/kafka/jumbo/broker.yaml
index 0a8111c..768635f 100644
--- a/hieradata/role/common/kafka/jumbo/broker.yaml
+++ b/hieradata/role/common/kafka/jumbo/broker.yaml
@@ -6,11 +6,11 @@
 # More info https://phabricator.wikimedia.org/T167304#3478277
 profile::kafka::broker::auth_acls_enabled: true
 
-# Enable SSL/TLS for Kafka brokers.  This requires
+# Enable SSL/TLS for Kafka.  This requires
 # that keystore and truststore files, and
 # profile::kafka::broker::ssl_password, are committed in
 # the expected location in ops/puppet/private.
-profile::kafka::broker::ssl_enabled: true
+profile::kafka::broker::ssl_enabled: false
 
 # Enable Monitoring (via Prometheus) and icinga alerts
 profile::kafka::broker::monitoring_enabled: true

-- 
To view, visit https://gerrit.wikimedia.org/r/394383
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I54a432ed9f512640ef34e6fc7adc9c29e5a4e99d
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ottomata <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to