{"action":"created","issue":{"url":"https://api.github.com/repos/ceph/ceph-deploy/issues/345","labels_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/345/labels{/name}","comments_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/345/comments","events_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/345/events","html_url":"https://github.com/ceph/ceph-deploy/pull/345","id":100538720,"number":345,"title":"check if ceph is installed","user":{"login":"alfredodeza","id":317847,"avatar_url":"https://avatars.githubusercontent.com/u/317847?v=3","gravatar_id":"","url":"https://api.github.com/users/alfredodeza","html_url":"https://github.com/alfredodeza","followers_url":"https://api.github.com/users/alfredodeza/followers","following_url":"https://api.github.com/users/alfredodeza/following{/other_user}","gists_url":"https://api.github.com/users/alfredodeza/gists{/gist_id}","starred_url":"https://api.github.com/users/alfredodeza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alfredodeza/subscriptions","organizations_url":"https://api.github.com/users/alfredodeza/orgs","repos_url":"https://api.github.com/users/alfredodeza/repos","events_url":"https://api.github.com/users/alfredodeza/events{/privacy}","received_events_url":"https://api.github.com/users/alfredodeza/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-08-12T13:04:48Z","updated_at":"2015-08-13T12:35:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ceph/ceph-deploy/pulls/345","html_url":"https://github.com/ceph/ceph-deploy/pull/345","diff_url":"https://github.com/ceph/ceph-deploy/pull/345.diff","patch_url":"https://github.com/ceph/ceph-deploy/pull/345.patch"},"body":"This is a PR with a bit of feature creep: the intent was to use the new `Ceph` object to provide an interface for versions and use it to map it against supported `systemd` versions.\r\n\r\nIt is not entirely sure if this approach will be used, but for now this can be used to fix (http://tracker.ceph.com/issues/11115) which allows to check if Ceph is installed and error accordingly. \r\n\r\nThe \"check if ceph is installed\" comes free with this implementation, if we decide later to not use versions we can change the approach.\r\n\r\nThis PR also moves the private function in `hosts/__init__.py` to `util.versions` because the functionality is now being used by more than one place. At some point the tests should map to the new location - but all tests pass as it is now."},"comment":{"url":"https://api.github.com/repos/ceph/ceph-deploy/issues/comments/130653206","html_url":"https://github.com/ceph/ceph-deploy/pull/345#issuecomment-130653206","issue_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/345","id":130653206,"user":{"login":"ceph-jenkins","id":5429108,"avatar_url":"https://avatars.githubusercontent.com/u/5429108?v=3","gravatar_id":"","url":"https://api.github.com/users/ceph-jenkins","html_url":"https://github.com/ceph-jenkins","followers_url":"https://api.github.com/users/ceph-jenkins/followers","following_url":"https://api.github.com/users/ceph-jenkins/following{/other_user}","gists_url":"https://api.github.com/users/ceph-jenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/ceph-jenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ceph-jenkins/subscriptions","organizations_url":"https://api.github.com/users/ceph-jenkins/orgs","repos_url":"https://api.github.com/users/ceph-jenkins/repos","events_url":"https://api.github.com/users/ceph-jenkins/events{/privacy}","received_events_url":"https://api.github.com/users/ceph-jenkins/received_events","type":"User","site_admin":false},"created_at":"2015-08-13T12:35:15Z","updated_at":"2015-08-13T12:35:15Z","body":"\nRefer to this link for build results (access rights to CI server needed): \nhttp://jenkins.ceph.com//job/ceph-deploy-pull-requests/1747/\nTest PASSed."}}