{"action":"created","issue":{"url":"https://api.github.com/repos/m-lab/npad-support/issues/19","repository_url":"https://api.github.com/repos/m-lab/npad-support","labels_url":"https://api.github.com/repos/m-lab/npad-support/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/m-lab/npad-support/issues/19/comments","events_url":"https://api.github.com/repos/m-lab/npad-support/issues/19/events","html_url":"https://github.com/m-lab/npad-support/pull/19","id":207384159,"number":19,"title":"Updates stop.sh to kill paris_rollins.py processes.","user":{"login":"nkinkade","id":1392825,"avatar_url":"https://avatars.githubusercontent.com/u/1392825?v=3","gravatar_id":"","url":"https://api.github.com/users/nkinkade","html_url":"https://github.com/nkinkade","followers_url":"https://api.github.com/users/nkinkade/followers","following_url":"https://api.github.com/users/nkinkade/following{/other_user}","gists_url":"https://api.github.com/users/nkinkade/gists{/gist_id}","starred_url":"https://api.github.com/users/nkinkade/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nkinkade/subscriptions","organizations_url":"https://api.github.com/users/nkinkade/orgs","repos_url":"https://api.github.com/users/nkinkade/repos","events_url":"https://api.github.com/users/nkinkade/events{/privacy}","received_events_url":"https://api.github.com/users/nkinkade/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-02-14T00:43:57Z","updated_at":"2017-02-14T00:52:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/m-lab/npad-support/pulls/19","html_url":"https://github.com/m-lab/npad-support/pull/19","diff_url":"https://github.com/m-lab/npad-support/pull/19.diff","patch_url":"https://github.com/m-lab/npad-support/pull/19.patch"},"body":"In addition to killing paris_rollins.py when stopped, this PR also starts and stops the httpd service in start.sh and stop.sh, respectively. On a clean boot up of an iupui_npad sliver httpd will be running anyway because the httpd service is configured to start on boot in initialize.sh, so adding `service httpd start` to start.sh is slightly redundant. However, it is good to have in there in case someone is manually starting/stopping httpd in the sliver, and because start.sh should probably be explicit about a hard dependency.\r\n\r\nStopping the httpd service in stop.sh seems good because it seems like when a sliver is stopped, all dependent processes should stop, and the sliver should be quiescent except for critical processes like crond and rsyncd, and not having httpd continue to server up NPAD web pages with links that will break because the underlying DiagServer isn't running."},"comment":{"url":"https://api.github.com/repos/m-lab/npad-support/issues/comments/279572798","html_url":"https://github.com/m-lab/npad-support/pull/19#issuecomment-279572798","issue_url":"https://api.github.com/repos/m-lab/npad-support/issues/19","id":279572798,"user":{"login":"stephen-soltesz","id":1085316,"avatar_url":"https://avatars.githubusercontent.com/u/1085316?v=3","gravatar_id":"","url":"https://api.github.com/users/stephen-soltesz","html_url":"https://github.com/stephen-soltesz","followers_url":"https://api.github.com/users/stephen-soltesz/followers","following_url":"https://api.github.com/users/stephen-soltesz/following{/other_user}","gists_url":"https://api.github.com/users/stephen-soltesz/gists{/gist_id}","starred_url":"https://api.github.com/users/stephen-soltesz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stephen-soltesz/subscriptions","organizations_url":"https://api.github.com/users/stephen-soltesz/orgs","repos_url":"https://api.github.com/users/stephen-soltesz/repos","events_url":"https://api.github.com/users/stephen-soltesz/events{/privacy}","received_events_url":"https://api.github.com/users/stephen-soltesz/received_events","type":"User","site_admin":false},"created_at":"2017-02-14T00:52:39Z","updated_at":"2017-02-14T00:52:39Z","body":"LGTM."}}
{"actor":{"display_login":"stephen-soltesz"}}