{"action":"created","issue":{"url":"https://api.github.com/repos/osiell/ansible-odoo/issues/38","repository_url":"https://api.github.com/repos/osiell/ansible-odoo","labels_url":"https://api.github.com/repos/osiell/ansible-odoo/issues/38/labels{/name}","comments_url":"https://api.github.com/repos/osiell/ansible-odoo/issues/38/comments","events_url":"https://api.github.com/repos/osiell/ansible-odoo/issues/38/events","html_url":"https://github.com/osiell/ansible-odoo/pull/38","id":174049685,"number":38,"title":"Fixes for ansible 2.1","user":{"login":"djowett","id":1470021,"avatar_url":"https://avatars.githubusercontent.com/u/1470021?v=3","gravatar_id":"","url":"https://api.github.com/users/djowett","html_url":"https://github.com/djowett","followers_url":"https://api.github.com/users/djowett/followers","following_url":"https://api.github.com/users/djowett/following{/other_user}","gists_url":"https://api.github.com/users/djowett/gists{/gist_id}","starred_url":"https://api.github.com/users/djowett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djowett/subscriptions","organizations_url":"https://api.github.com/users/djowett/orgs","repos_url":"https://api.github.com/users/djowett/repos","events_url":"https://api.github.com/users/djowett/events{/privacy}","received_events_url":"https://api.github.com/users/djowett/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-08-30T15:24:57Z","updated_at":"2016-08-30T15:40:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/osiell/ansible-odoo/pulls/38","html_url":"https://github.com/osiell/ansible-odoo/pull/38","diff_url":"https://github.com/osiell/ansible-odoo/pull/38.diff","patch_url":"https://github.com/osiell/ansible-odoo/pull/38.patch"},"body":"Includes fix for #35 and #37 - the later needs testing with remote databases (which I don't use).\r\n\r\nPlease merge with care! And come back with any questions!\r\n\r\ncc @sebalix @Neobis-Ludo\r\n\r\nThe following pages were helpful in building fix for #37\r\n\r\n* http://stackoverflow.com/questions/37388696/ansible-conditionally-delegate-to-local-or-remote\r\n* http://docs.ansible.com/ansible/playbooks_delegation.html#delegation"},"comment":{"url":"https://api.github.com/repos/osiell/ansible-odoo/issues/comments/243482880","html_url":"https://github.com/osiell/ansible-odoo/pull/38#issuecomment-243482880","issue_url":"https://api.github.com/repos/osiell/ansible-odoo/issues/38","id":243482880,"user":{"login":"sebalix","id":5315285,"avatar_url":"https://avatars.githubusercontent.com/u/5315285?v=3","gravatar_id":"","url":"https://api.github.com/users/sebalix","html_url":"https://github.com/sebalix","followers_url":"https://api.github.com/users/sebalix/followers","following_url":"https://api.github.com/users/sebalix/following{/other_user}","gists_url":"https://api.github.com/users/sebalix/gists{/gist_id}","starred_url":"https://api.github.com/users/sebalix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sebalix/subscriptions","organizations_url":"https://api.github.com/users/sebalix/orgs","repos_url":"https://api.github.com/users/sebalix/repos","events_url":"https://api.github.com/users/sebalix/events{/privacy}","received_events_url":"https://api.github.com/users/sebalix/received_events","type":"User","site_admin":false},"created_at":"2016-08-30T15:40:30Z","updated_at":"2016-08-30T15:40:30Z","body":"Thank you @djowett for these fixes. The PostgreSQL part seems really elegant, and avoid code/task duplication. I need to do some tests, especially regarding the use of `delegate_to` even for a local PostgreSQL.\r\nIt is OK if we need to pin Ansible 2.1 exclusively, it seems hard to keep the compatibility with older versions.\r\nAlso, Travis does not seem happy regarding the `ansible_host` variable."}}
{"actor":{"display_login":"sebalix"}}