Dzahn has submitted this change and it was merged.

Change subject: ganglia: fix systemd instance service name
......................................................................


ganglia: fix systemd instance service name

This was the wrong format, the puppet systemd provider tries to
exec this as systemctl start ganglia-monitor-aggregator-instance-2027,
etc.

The format should be:

systemctl start [email protected]

to evoke an instance from the @template

Bug:T124197
Change-Id: Iaf53dc32838d7c8e3ea74ab4fa819f584df0b8fd
---
M modules/ganglia/manifests/monitor/aggregator/instance.pp
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/modules/ganglia/manifests/monitor/aggregator/instance.pp 
b/modules/ganglia/manifests/monitor/aggregator/instance.pp
index c2cd58d..5a6e25f 100644
--- a/modules/ganglia/manifests/monitor/aggregator/instance.pp
+++ b/modules/ganglia/manifests/monitor/aggregator/instance.pp
@@ -47,7 +47,7 @@
     # on jessie/systemd each instance is a separate service
     # which is spawned from a common service template
     if os_version('debian >= jessie') {
-        service { "ganglia-monitor-aggregator-instance-${id}":
+        service { "ganglia-monitor-aggregator@${id}.service":
             ensure   => running,
             provider => systemd,
             enable   => true,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf53dc32838d7c8e3ea74ab4fa819f584df0b8fd
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to