{"action":"created","issue":{"url":"https://api.github.com/repos/coreos/fleet/issues/1566","repository_url":"https://api.github.com/repos/coreos/fleet","labels_url":"https://api.github.com/repos/coreos/fleet/issues/1566/labels{/name}","comments_url":"https://api.github.com/repos/coreos/fleet/issues/1566/comments","events_url":"https://api.github.com/repos/coreos/fleet/issues/1566/events","html_url":"https://github.com/coreos/fleet/pull/1566","id":150895999,"number":1566,"title":"fleetctl: check also systemd states after starting units","user":{"login":"dongsupark","id":10096906,"avatar_url":"https://avatars.githubusercontent.com/u/10096906?v=3","gravatar_id":"","url":"https://api.github.com/users/dongsupark","html_url":"https://github.com/dongsupark","followers_url":"https://api.github.com/users/dongsupark/followers","following_url":"https://api.github.com/users/dongsupark/following{/other_user}","gists_url":"https://api.github.com/users/dongsupark/gists{/gist_id}","starred_url":"https://api.github.com/users/dongsupark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongsupark/subscriptions","organizations_url":"https://api.github.com/users/dongsupark/orgs","repos_url":"https://api.github.com/users/dongsupark/repos","events_url":"https://api.github.com/users/dongsupark/events{/privacy}","received_events_url":"https://api.github.com/users/dongsupark/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/coreos/fleet/labels/area/usability","name":"area/usability","color":"bfe5bf"},{"url":"https://api.github.com/repos/coreos/fleet/labels/component/fleetctl","name":"component/fleetctl","color":"0052cc"},{"url":"https://api.github.com/repos/coreos/fleet/labels/priority/P2","name":"priority/P2","color":"fbca04"}],"state":"closed","locked":false,"assignee":{"login":"dongsupark","id":10096906,"avatar_url":"https://avatars.githubusercontent.com/u/10096906?v=3","gravatar_id":"","url":"https://api.github.com/users/dongsupark","html_url":"https://github.com/dongsupark","followers_url":"https://api.github.com/users/dongsupark/followers","following_url":"https://api.github.com/users/dongsupark/following{/other_user}","gists_url":"https://api.github.com/users/dongsupark/gists{/gist_id}","starred_url":"https://api.github.com/users/dongsupark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongsupark/subscriptions","organizations_url":"https://api.github.com/users/dongsupark/orgs","repos_url":"https://api.github.com/users/dongsupark/repos","events_url":"https://api.github.com/users/dongsupark/events{/privacy}","received_events_url":"https://api.github.com/users/dongsupark/received_events","type":"User","site_admin":false},"assignees":[{"login":"dongsupark","id":10096906,"avatar_url":"https://avatars.githubusercontent.com/u/10096906?v=3","gravatar_id":"","url":"https://api.github.com/users/dongsupark","html_url":"https://github.com/dongsupark","followers_url":"https://api.github.com/users/dongsupark/followers","following_url":"https://api.github.com/users/dongsupark/following{/other_user}","gists_url":"https://api.github.com/users/dongsupark/gists{/gist_id}","starred_url":"https://api.github.com/users/dongsupark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongsupark/subscriptions","organizations_url":"https://api.github.com/users/dongsupark/orgs","repos_url":"https://api.github.com/users/dongsupark/repos","events_url":"https://api.github.com/users/dongsupark/events{/privacy}","received_events_url":"https://api.github.com/users/dongsupark/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2016-04-25T15:18:18Z","updated_at":"2016-08-19T08:28:25Z","closed_at":"2016-07-22T09:48:21Z","pull_request":{"url":"https://api.github.com/repos/coreos/fleet/pulls/1566","html_url":"https://github.com/coreos/fleet/pull/1566","diff_url":"https://github.com/coreos/fleet/pull/1566.diff","patch_url":"https://github.com/coreos/fleet/pull/1566.patch"},"body":"To resolve inconsistency between ``\"systemctl start\"`` and ``\"fleetctl start\"``, fleetctl needs to check systemd states after starting units. If the systemd state is not a desired one, wait until it gets an active state.\r\n\r\nFixes: https://github.com/coreos/fleet/issues/1025\r\n"},"comment":{"url":"https://api.github.com/repos/coreos/fleet/issues/comments/240959339","html_url":"https://github.com/coreos/fleet/pull/1566#issuecomment-240959339","issue_url":"https://api.github.com/repos/coreos/fleet/issues/1566","id":240959339,"user":{"login":"dongsupark","id":10096906,"avatar_url":"https://avatars.githubusercontent.com/u/10096906?v=3","gravatar_id":"","url":"https://api.github.com/users/dongsupark","html_url":"https://github.com/dongsupark","followers_url":"https://api.github.com/users/dongsupark/followers","following_url":"https://api.github.com/users/dongsupark/following{/other_user}","gists_url":"https://api.github.com/users/dongsupark/gists{/gist_id}","starred_url":"https://api.github.com/users/dongsupark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongsupark/subscriptions","organizations_url":"https://api.github.com/users/dongsupark/orgs","repos_url":"https://api.github.com/users/dongsupark/repos","events_url":"https://api.github.com/users/dongsupark/events{/privacy}","received_events_url":"https://api.github.com/users/dongsupark/received_events","type":"User","site_admin":false},"created_at":"2016-08-19T08:28:25Z","updated_at":"2016-08-19T08:28:25Z","body":"@jonboulle Ok. I'll try to take a different approach as you suggested. From my understanding, `schema.UnitState` has already its own fields like `SystemdActiveState`, `SystemdLoadState`, and `SystemdSubState`. So we should make use of `cAPI.UnitStates()` to retrieve these states, to make sure that these are the expected states. Is that correct?"}}
{"actor":{"display_login":"dongsupark"}}