{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/2208","repository_url":"https://api.github.com/repos/ansible/ansible-modules-extras","labels_url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/2208/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/2208/comments","events_url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/2208/events","html_url":"https://github.com/ansible/ansible-modules-extras/pull/2208","id":154042070,"number":2208,"title":"Add lxd_container module","user":{"login":"hnakamur","id":19299,"avatar_url":"https://avatars.githubusercontent.com/u/19299?v=3","gravatar_id":"","url":"https://api.github.com/users/hnakamur","html_url":"https://github.com/hnakamur","followers_url":"https://api.github.com/users/hnakamur/followers","following_url":"https://api.github.com/users/hnakamur/following{/other_user}","gists_url":"https://api.github.com/users/hnakamur/gists{/gist_id}","starred_url":"https://api.github.com/users/hnakamur/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hnakamur/subscriptions","organizations_url":"https://api.github.com/users/hnakamur/orgs","repos_url":"https://api.github.com/users/hnakamur/repos","events_url":"https://api.github.com/users/hnakamur/events{/privacy}","received_events_url":"https://api.github.com/users/hnakamur/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/labels/cloud","name":"cloud","color":"c7def8"},{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/labels/needs_revision","name":"needs_revision","color":"eb6420"},{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/labels/new_plugin","name":"new_plugin","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":32,"created_at":"2016-05-10T15:53:03Z","updated_at":"2016-05-18T17:08:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/pulls/2208","html_url":"https://github.com/ansible/ansible-modules-extras/pull/2208","diff_url":"https://github.com/ansible/ansible-modules-extras/pull/2208.diff","patch_url":"https://github.com/ansible/ansible-modules-extras/pull/2208.patch"},"body":"##### ISSUE TYPE\r\n - New Module Pull Request\r\n\r\n##### COMPONENT NAME\r\nlxd_container\r\n\r\n##### ANSIBLE VERSION\r\n```\r\nansible 2.2.0\r\n  config file = /etc/ansible/ansible.cfg\r\n  configured module search path = Default w/o overrides\r\n```\r\n\r\n##### SUMMARY\r\nA new module to manage LXD Containers.\r\nPlease review it and give me feedbacks.\r\nThanks!\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/comments/220094286","html_url":"https://github.com/ansible/ansible-modules-extras/pull/2208#issuecomment-220094286","issue_url":"https://api.github.com/repos/ansible/ansible-modules-extras/issues/2208","id":220094286,"user":{"login":"candlerb","id":44789,"avatar_url":"https://avatars.githubusercontent.com/u/44789?v=3","gravatar_id":"","url":"https://api.github.com/users/candlerb","html_url":"https://github.com/candlerb","followers_url":"https://api.github.com/users/candlerb/followers","following_url":"https://api.github.com/users/candlerb/following{/other_user}","gists_url":"https://api.github.com/users/candlerb/gists{/gist_id}","starred_url":"https://api.github.com/users/candlerb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/candlerb/subscriptions","organizations_url":"https://api.github.com/users/candlerb/orgs","repos_url":"https://api.github.com/users/candlerb/repos","events_url":"https://api.github.com/users/candlerb/events{/privacy}","received_events_url":"https://api.github.com/users/candlerb/received_events","type":"User","site_admin":false},"created_at":"2016-05-18T17:08:52Z","updated_at":"2016-05-18T17:08:52Z","body":"> So what is the module state of \"restarted\".  Is it not a state, it is a command, which should result in the state of \"running\"\r\n\r\nThis usage is very consistent with the rest of ansible, especially the core [service](http://docs.ansible.com/ansible/service_module.html) module: e.g. `service name=apache2 state=restarted`\r\n\r\n`state=restarted` means \"I want it to be running, but if it was already running, restart it\"\r\n\r\nAs for \"running\" versus \"started\": again \"started\" is consistent with the service module. My vote would be for ansible to be consistent internally as far as possible.\r\n\r\n(Clearly it cannot be 100% consistent: e.g. the service module has no idea of a service being \"absent\", because that state applies to the package not the service. This is necessary, since one package can provide multiple services, and you may want to control those services separately)"}}