{"action":"created","issue":{"url":"https://api.github.com/repos/puppetlabs/puppet/issues/3700","labels_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3700/labels{/name}","comments_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3700/comments","events_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3700/events","html_url":"https://github.com/puppetlabs/puppet/pull/3700","id":60573157,"number":3700,"title":"(PUP-3931) Change systemd KillMode to process for agent","user":{"login":"shanemadden","id":3057436,"avatar_url":"https://avatars.githubusercontent.com/u/3057436?v=3","gravatar_id":"","url":"https://api.github.com/users/shanemadden","html_url":"https://github.com/shanemadden","followers_url":"https://api.github.com/users/shanemadden/followers","following_url":"https://api.github.com/users/shanemadden/following{/other_user}","gists_url":"https://api.github.com/users/shanemadden/gists{/gist_id}","starred_url":"https://api.github.com/users/shanemadden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shanemadden/subscriptions","organizations_url":"https://api.github.com/users/shanemadden/orgs","repos_url":"https://api.github.com/users/shanemadden/repos","events_url":"https://api.github.com/users/shanemadden/events{/privacy}","received_events_url":"https://api.github.com/users/shanemadden/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/puppetlabs/puppet/labels/Community","name":"Community","color":"eb6420"},{"url":"https://api.github.com/repos/puppetlabs/puppet/labels/Merge%20After%20Unfreeze","name":"Merge After Unfreeze","color":"207de5"},{"url":"https://api.github.com/repos/puppetlabs/puppet/labels/Triaged","name":"Triaged","color":"fef2c0"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-03-10T21:33:07Z","updated_at":"2015-04-30T22:53:40Z","closed_at":"2015-03-18T17:41:07Z","pull_request":{"url":"https://api.github.com/repos/puppetlabs/puppet/pulls/3700","html_url":"https://github.com/puppetlabs/puppet/pull/3700","diff_url":"https://github.com/puppetlabs/puppet/pull/3700.diff","patch_url":"https://github.com/puppetlabs/puppet/pull/3700.patch"},"body":"Currently, the entire control group is sent a kill; if a puppet agent run under the control group is running an upgrade of the puppet package, both the running agent process and the yum process doing the upgrade are killed as well as the daemon agent process, leaving the puppet installation in a bad state.  This brings the behavior in line with the behavior under the SysV init script."},"comment":{"url":"https://api.github.com/repos/puppetlabs/puppet/issues/comments/97993201","html_url":"https://github.com/puppetlabs/puppet/pull/3700#issuecomment-97993201","issue_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3700","id":97993201,"user":{"login":"opoplawski","id":814662,"avatar_url":"https://avatars.githubusercontent.com/u/814662?v=3","gravatar_id":"","url":"https://api.github.com/users/opoplawski","html_url":"https://github.com/opoplawski","followers_url":"https://api.github.com/users/opoplawski/followers","following_url":"https://api.github.com/users/opoplawski/following{/other_user}","gists_url":"https://api.github.com/users/opoplawski/gists{/gist_id}","starred_url":"https://api.github.com/users/opoplawski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/opoplawski/subscriptions","organizations_url":"https://api.github.com/users/opoplawski/orgs","repos_url":"https://api.github.com/users/opoplawski/repos","events_url":"https://api.github.com/users/opoplawski/events{/privacy}","received_events_url":"https://api.github.com/users/opoplawski/received_events","type":"User","site_admin":false},"created_at":"2015-04-30T22:53:39Z","updated_at":"2015-04-30T22:53:39Z","body":"The problem with this now is that a simple systemctl restart puppet fails because the child process refuses to die:\r\n~~~~~\r\n# ps -fe | grep puppet\r\nroot     21421     1  0 16:45 ?        00:00:00 /bin/sh /usr/bin/start-puppet-agent agent  --no-daemonize\r\nroot     21424 21421  0 16:45 ?        00:00:01 /usr/bin/ruby-mri /usr/bin/puppet agent --no-daemonize\r\nroot     22536  2094  0 16:51 pts/4    00:00:00 grep --color=auto puppet\r\n# systemctl restart puppet\r\n# ps -fe | grep puppet\r\nroot     21424     1  0 16:45 ?        00:00:01 /usr/bin/ruby-mri /usr/bin/puppet agent --no-daemonize\r\n~~~~~\r\nAt least for me on Fedora 21."}}