{"action":"created","issue":{"url":"https://api.github.com/repos/ceph/ceph-deploy/issues/286","labels_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/286/labels{/name}","comments_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/286/comments","events_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/286/events","html_url":"https://github.com/ceph/ceph-deploy/pull/286","id":75644180,"number":286,"title":"MDS: systemd enable","user":{"login":"goldwynr","id":2651714,"avatar_url":"https://avatars.githubusercontent.com/u/2651714?v=3","gravatar_id":"","url":"https://api.github.com/users/goldwynr","html_url":"https://github.com/goldwynr","followers_url":"https://api.github.com/users/goldwynr/followers","following_url":"https://api.github.com/users/goldwynr/following{/other_user}","gists_url":"https://api.github.com/users/goldwynr/gists{/gist_id}","starred_url":"https://api.github.com/users/goldwynr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/goldwynr/subscriptions","organizations_url":"https://api.github.com/users/goldwynr/orgs","repos_url":"https://api.github.com/users/goldwynr/repos","events_url":"https://api.github.com/users/goldwynr/events{/privacy}","received_events_url":"https://api.github.com/users/goldwynr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-12T15:39:11Z","updated_at":"2015-05-12T15:54:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ceph/ceph-deploy/pulls/286","html_url":"https://github.com/ceph/ceph-deploy/pull/286","diff_url":"https://github.com/ceph/ceph-deploy/pull/286.diff","patch_url":"https://github.com/ceph/ceph-deploy/pull/286.patch"},"body":"MDS is not enabled if the distribution uses systemd. Add support for distros using systemd.\r\nAlso fixed a directory creation fail in case of a new installation while create."},"comment":{"url":"https://api.github.com/repos/ceph/ceph-deploy/issues/comments/101329353","html_url":"https://github.com/ceph/ceph-deploy/pull/286#issuecomment-101329353","issue_url":"https://api.github.com/repos/ceph/ceph-deploy/issues/286","id":101329353,"user":{"login":"trhoden","id":1627176,"avatar_url":"https://avatars.githubusercontent.com/u/1627176?v=3","gravatar_id":"","url":"https://api.github.com/users/trhoden","html_url":"https://github.com/trhoden","followers_url":"https://api.github.com/users/trhoden/followers","following_url":"https://api.github.com/users/trhoden/following{/other_user}","gists_url":"https://api.github.com/users/trhoden/gists{/gist_id}","starred_url":"https://api.github.com/users/trhoden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/trhoden/subscriptions","organizations_url":"https://api.github.com/users/trhoden/orgs","repos_url":"https://api.github.com/users/trhoden/repos","events_url":"https://api.github.com/users/trhoden/events{/privacy}","received_events_url":"https://api.github.com/users/trhoden/received_events","type":"User","site_admin":false},"created_at":"2015-05-12T15:54:52Z","updated_at":"2015-05-12T15:54:52Z","body":"@goldwynr Can you elaborate on the mkdir piece a bit more for me?\r\n\r\n/var/lib/ceph/mds is owned by the \"ceph\" package, and should be there already whenever ceph is installed.  We should never have to \"make\" /var/lib/ceph/mds.  However, it may be that this was a recent change?  What version of Ceph did you run into this with?\r\n\r\nFor example, I recently ran into the exact same thing with RGW.  I fixed that in https://github.com/ceph/ceph-deploy/pull/283.  So it's not that I am opposed to the fix, just want to make sure I understand.\r\n\r\nIn that referenced PR, a new \"safe_mkdirs()\" method was introduced.  https://github.com/ceph/ceph-deploy/commit/4bc9e271d5d5d7715dfd36d0142e8aba2911138e\r\n\r\nI would use that instead method instead."}}