Giuseppe Lavagetto has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/371480 )

Change subject: confluent: convert to systemd::service
......................................................................


confluent: convert to systemd::service

Bug: T173078
Change-Id: I161e43a2993dd12e2783b981b17bfcc75442d1e9
---
M modules/confluent/manifests/kafka/broker.pp
M modules/confluent/manifests/kafka/mirror/instance.pp
2 files changed, 6 insertions(+), 9 deletions(-)

Approvals:
  Giuseppe Lavagetto: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/confluent/manifests/kafka/broker.pp 
b/modules/confluent/manifests/kafka/broker.pp
index a6ca126..99a7203 100644
--- a/modules/confluent/manifests/kafka/broker.pp
+++ b/modules/confluent/manifests/kafka/broker.pp
@@ -354,10 +354,9 @@
     # We don't want to subscribe to the config files here.
     # It will be better to manually restart Kafka when
     # the config files changes.
-    base::service_unit{ 'kafka':
+    systemd::service { 'kafka':
         ensure  => $service_ensure,
-        systemd => true,
-        refresh => false,
+        content => systemd_template('kafka'),
         require => [
             File[$log_dirs],
             File['/etc/kafka/server.properties'],
diff --git a/modules/confluent/manifests/kafka/mirror/instance.pp 
b/modules/confluent/manifests/kafka/mirror/instance.pp
index 11f7758..1a478ae 100644
--- a/modules/confluent/manifests/kafka/mirror/instance.pp
+++ b/modules/confluent/manifests/kafka/mirror/instance.pp
@@ -152,12 +152,10 @@
     }
     # Start the MirrorMaker instance.
     # We don't want to subscribe to the config files here.
-    base::service_unit{ "kafka-mirror-${mirror_name}":
-        ensure        => $service_ensure,
-        template_name => 'kafka-mirror',
-        systemd       => true,
-        refresh       => false,
-        require       => [
+    systemd::service { "kafka-mirror-${mirror_name}":
+        ensure  => $service_ensure,
+        content => systemd_template('kafka-mirror'),
+        require => [
             File["/etc/kafka/mirror/${mirror_name}/log4j.properties"],
             File["/etc/kafka/mirror/${mirror_name}/consumer.properties"],
             File["/etc/kafka/mirror/${mirror_name}/producer.properties"],

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I161e43a2993dd12e2783b981b17bfcc75442d1e9
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <glavage...@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

Reply via email to