ArielGlenn has submitted this change and it was merged.

Change subject: restart of salt minion should not kill all subprocesses
......................................................................


restart of salt minion should not kill all subprocesses

this fixes a problem with installs on jessie which used to leave
salt minion upgrades run via salt from the master in a broken
half-installed state.

Change-Id: I39494dc6e3897a0eb6b576850b42db2f7c162b1b
---
M modules/salt/manifests/minion.pp
1 file changed, 15 insertions(+), 0 deletions(-)

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



diff --git a/modules/salt/manifests/minion.pp b/modules/salt/manifests/minion.pp
index d186c9f..e27c02d 100644
--- a/modules/salt/manifests/minion.pp
+++ b/modules/salt/manifests/minion.pp
@@ -116,4 +116,19 @@
         mode   => '0444',
     }
 
+    if $::initsystem == 'systemd' {
+        file { '/etc/systemd/system/salt-minion.service.d/':
+            ensure => directory,
+            owner  => 'root',
+            group  => 'root',
+            mode   => '0755',
+        }
+
+        file { '/etc/systemd/system/salt-minion.service.d/killmode.conf':
+            content => '[Service]\nKillMode=process\n',
+            owner   => 'root',
+            group   => 'root',
+            mode    => '0444',
+        }
+    }
 }

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

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

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

Reply via email to