Yuvipanda has submitted this change and it was merged.

Change subject: labstore: Fix boolean conditional to do actual needed check
......................................................................


labstore: Fix boolean conditional to do actual needed check

Change-Id: I2f16568550775fe8843912287008f93124e6716e
---
M modules/base/manifests/service_unit.pp
1 file changed, 6 insertions(+), 4 deletions(-)

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



diff --git a/modules/base/manifests/service_unit.pp 
b/modules/base/manifests/service_unit.pp
index ba52ac2..278b4ae 100644
--- a/modules/base/manifests/service_unit.pp
+++ b/modules/base/manifests/service_unit.pp
@@ -103,10 +103,12 @@
             group   => root,
         }
 
-        if $refresh and $declare_service {
-            File[$path] ~> Service[$name]
-        } else {
-            File[$path] -> Service[$name]
+        if $declare_service {
+            if $refresh {
+                File[$path] ~> Service[$name]
+            } else {
+                File[$path] -> Service[$name]
+            }
         }
 
         if $::initsystem == 'systemd' {

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

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

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

Reply via email to