{"action":"created","issue":{"url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common/issues/47","repository_url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common","labels_url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common/issues/47/labels{/name}","comments_url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common/issues/47/comments","events_url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common/issues/47/events","html_url":"https://github.com/chef-cookbooks/enterprise-chef-common/pull/47","id":158247474,"number":47,"title":"Install unit files in `/etc/systemd/system`","user":{"login":"yzl","id":719439,"avatar_url":"https://avatars.githubusercontent.com/u/719439?v=3","gravatar_id":"","url":"https://api.github.com/users/yzl","html_url":"https://github.com/yzl","followers_url":"https://api.github.com/users/yzl/followers","following_url":"https://api.github.com/users/yzl/following{/other_user}","gists_url":"https://api.github.com/users/yzl/gists{/gist_id}","starred_url":"https://api.github.com/users/yzl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yzl/subscriptions","organizations_url":"https://api.github.com/users/yzl/orgs","repos_url":"https://api.github.com/users/yzl/repos","events_url":"https://api.github.com/users/yzl/events{/privacy}","received_events_url":"https://api.github.com/users/yzl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-06-02T21:48:04Z","updated_at":"2016-06-06T15:41:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common/pulls/47","html_url":"https://github.com/chef-cookbooks/enterprise-chef-common/pull/47","diff_url":"https://github.com/chef-cookbooks/enterprise-chef-common/pull/47.diff","patch_url":"https://github.com/chef-cookbooks/enterprise-chef-common/pull/47.patch"},"body":"### Description\r\nInstall unit files in `/etc/systemd/system`. `systemd` gives this location priority.\r\n\r\n### Issues Resolved\r\n[chef-server #572](https://github.com/chef/chef-server/issues/572)\r\n\r\n### Check List\r\n- [x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD\r\n- [x] New functionality includes testing.\r\n- [ ] New functionality has been documented in the README if applicable\r\n- [x] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common/issues/comments/223998454","html_url":"https://github.com/chef-cookbooks/enterprise-chef-common/pull/47#issuecomment-223998454","issue_url":"https://api.github.com/repos/chef-cookbooks/enterprise-chef-common/issues/47","id":223998454,"user":{"login":"mmzyk","id":66784,"avatar_url":"https://avatars.githubusercontent.com/u/66784?v=3","gravatar_id":"","url":"https://api.github.com/users/mmzyk","html_url":"https://github.com/mmzyk","followers_url":"https://api.github.com/users/mmzyk/followers","following_url":"https://api.github.com/users/mmzyk/following{/other_user}","gists_url":"https://api.github.com/users/mmzyk/gists{/gist_id}","starred_url":"https://api.github.com/users/mmzyk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmzyk/subscriptions","organizations_url":"https://api.github.com/users/mmzyk/orgs","repos_url":"https://api.github.com/users/mmzyk/repos","events_url":"https://api.github.com/users/mmzyk/events{/privacy}","received_events_url":"https://api.github.com/users/mmzyk/received_events","type":"User","site_admin":false},"created_at":"2016-06-06T15:41:38Z","updated_at":"2016-06-06T15:41:38Z","body":"@marcparadise @stevendanna I initially was thinking like y'all - we should remove the old one, but how likely are we going to be on a system that we're installing to the new location with systemd and the old location exists? Wouldn't this just be failed installs?\r\n\r\nNow that I think about it, this would be RHEL 7, given it doesn't have the compilation option that necessitates this fix for Ubuntu. So given that, it would be nice to clean up the old files in a situation like that, @yzl. It'll make for easier debugging if we don't have misleading files left on the system, should we ever be in the situation we're debugging this. I didn't follow your logic for why you left the files in place though @yzl, so maybe I'm missing understanding.\r\n\r\nOverall, +1 and think this is fine to go down and the clean up can always be a separate PR.  "}}
{"actor":{"display_login":"mmzyk"}}