Giuseppe Lavagetto has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/242578

Change subject: kube2proxy: fix systemd declaration
......................................................................

kube2proxy: fix systemd declaration

Change-Id: Ia06fb84a6b582338107c778c19d89871db99b427
---
M modules/toollabs/manifests/kube2proxy.pp
1 file changed, 19 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/78/242578/1

diff --git a/modules/toollabs/manifests/kube2proxy.pp 
b/modules/toollabs/manifests/kube2proxy.pp
index bc0252d..931df5b 100644
--- a/modules/toollabs/manifests/kube2proxy.pp
+++ b/modules/toollabs/manifests/kube2proxy.pp
@@ -40,13 +40,26 @@
     # instead of having processes syncing every proxy
     # with kubernetes is a bad idea, we're doing it just
     # because that's how OGE integration worked.
-    $should_run = ($::hostname != $active_proxy)
+    if service_ensure($ensure) == 'running' {
+        $should_run = hiera('active_proxy') ?{
+            $::hostname => 'running',
+            default     => 'stopped'
+        }
+        $params = {'ensure' => $should_run}
+    } else {
+        $params = {}
+    }
+
     base::service_unit{ 'kubesync':
-        ensure    => $should_run,
-        refresh   => true,
-        systemd   => true,
-        upstart   => true,
-        subscribe => 
[File['/usr/local/sbin/kube2proxy'],File['/var/lib/kubernetes/ssl/certs/ca.pem']],
+        ensure         => $ensure,
+        refresh        => true,
+        systemd        => true,
+        upstart        => true,
+        service_params => $params,
+        subscribe => [
+                      File['/usr/local/sbin/kube2proxy'],
+                      File['/var/lib/kubernetes/ssl/certs/ca.pem']
+                      ],
     }
 
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia06fb84a6b582338107c778c19d89871db99b427
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <glavage...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to