{"action":"created","issue":{"url":"https://api.github.com/repos/strongloop/strong-supervisor/issues/137","labels_url":"https://api.github.com/repos/strongloop/strong-supervisor/issues/137/labels{/name}","comments_url":"https://api.github.com/repos/strongloop/strong-supervisor/issues/137/comments","events_url":"https://api.github.com/repos/strongloop/strong-supervisor/issues/137/events","html_url":"https://github.com/strongloop/strong-supervisor/issues/137","id":88594920,"number":137,"title":"`sl-runctl stop` returns success before the supervisor is stopped","user":{"login":"danielwhite","id":481511,"avatar_url":"https://avatars.githubusercontent.com/u/481511?v=3","gravatar_id":"","url":"https://api.github.com/users/danielwhite","html_url":"https://github.com/danielwhite","followers_url":"https://api.github.com/users/danielwhite/followers","following_url":"https://api.github.com/users/danielwhite/following{/other_user}","gists_url":"https://api.github.com/users/danielwhite/gists{/gist_id}","starred_url":"https://api.github.com/users/danielwhite/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielwhite/subscriptions","organizations_url":"https://api.github.com/users/danielwhite/orgs","repos_url":"https://api.github.com/users/danielwhite/repos","events_url":"https://api.github.com/users/danielwhite/events{/privacy}","received_events_url":"https://api.github.com/users/danielwhite/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/strongloop/strong-supervisor/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-06-16T02:37:02Z","updated_at":"2015-06-19T20:44:04Z","closed_at":null,"body":"The fundamental problem behind issue #136 was the use of `stop` and `start` in quick succession, where a running supervisor was seen as a successful start.\r\n\r\nIt would be easier to script against if the `stop` CLI command only returns once the master is actually stopped.\r\n\r\nTo reproduce (assuming a running process):\r\n\r\n```\r\n$ sl-runctl stop && sl-runctl status\r\n```\r\n\r\nIf the problem exists, then the supervisor information is returned. For example:\r\n\r\n```\r\nmaster pid: 28943\r\nworker count: 1\r\nworker id 6: { pid: 28964, uptime: 1445, startTime: 1434419524881 }\r\n```\r\n\r\nIf the problem is resolved, then there should be an indication that the master is stopped. For example:\r\n\r\n```\r\nCommunication error (connect ECONNREFUSED), check master is listening\r\n```\r\n\r\nA fix didn't seem trivial, so I might not get a chance to fix this in the near future, so suggestions would be welcome."},"comment":{"url":"https://api.github.com/repos/strongloop/strong-supervisor/issues/comments/113637988","html_url":"https://github.com/strongloop/strong-supervisor/issues/137#issuecomment-113637988","issue_url":"https://api.github.com/repos/strongloop/strong-supervisor/issues/137","id":113637988,"user":{"login":"sam-github","id":17607,"avatar_url":"https://avatars.githubusercontent.com/u/17607?v=3","gravatar_id":"","url":"https://api.github.com/users/sam-github","html_url":"https://github.com/sam-github","followers_url":"https://api.github.com/users/sam-github/followers","following_url":"https://api.github.com/users/sam-github/following{/other_user}","gists_url":"https://api.github.com/users/sam-github/gists{/gist_id}","starred_url":"https://api.github.com/users/sam-github/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sam-github/subscriptions","organizations_url":"https://api.github.com/users/sam-github/orgs","repos_url":"https://api.github.com/users/sam-github/repos","events_url":"https://api.github.com/users/sam-github/events{/privacy}","received_events_url":"https://api.github.com/users/sam-github/received_events","type":"User","site_admin":false},"created_at":"2015-06-19T20:44:04Z","updated_at":"2015-06-19T20:44:04Z","body":"If you want the app and its runner packaged together, you'd need something like `slc pm server/server.js` - that's what I mean by one-shot.\r\n\r\nYou have a heavily rpm-based deploy, you need the app to be packaged in the rpm, using `slc depoy` is not an option?"}}