)]}'
{"id":"openstack%2Fopenstack-ansible~461012","triplet_id":"openstack%2Fopenstack-ansible~master~I1399b6727ec8894dd3b9c464f6addbeea1c1f4f9","project":"openstack/openstack-ansible","branch":"master","topic":"no-ssh-ping","hashtags":[],"change_id":"I1399b6727ec8894dd3b9c464f6addbeea1c1f4f9","subject":"Use wait_for_connection instead of wait_for to check container","status":"MERGED","created":"2017-04-28 12:18:02.000000000","updated":"2017-04-28 14:54:16.000000000","submitted":"2017-04-28 14:54:16.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0a6824841dbff0bc37d6a716c2f01b261d7c366b","_number":461012,"virtual_id_number":461012,"owner":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2017-04-28 14:54:16.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"value":0,"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2017-04-28 12:19:11.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"value":2,"date":"2017-04-28 12:20:11.000000000","_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"value":1,"date":"2017-04-28 12:20:11.000000000","_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"value":0,"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-04-28 12:19:11.000000000","updated_by":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"reviewer":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"state":"REVIEWER"},{"updated":"2017-04-28 12:20:11.000000000","updated_by":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"reviewer":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"state":"REVIEWER"},{"updated":"2017-04-28 14:54:16.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"aff63fce8993da815b90b717077b315302c1c2ba","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2017-04-28 12:18:02.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"938d83c62cfe83ddedba073af5e188160a49ce4b","author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"date":"2017-04-28 12:19:11.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"8c8a2cc9d94c43ebce414e4b4a287f023e55982d","author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"date":"2017-04-28 12:20:11.000000000","message":"Patch Set 1: Workflow+1 Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"ce20105d63db33fd8c6a0dd5e345e434e2fa6f9b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-04-28 13:37:41.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-openstack-ansible-docs-ubuntu-xenial http://docs-draft.openstack.org/12/461012/1/check/gate-openstack-ansible-docs-ubuntu-xenial/cef3823//doc/build/html/ : SUCCESS in 50s\n- gate-openstack-ansible-linters-ubuntu-xenial http://logs.openstack.org/12/461012/1/check/gate-openstack-ansible-linters-ubuntu-xenial/588e9d3/ : SUCCESS in 8m 01s\n- gate-openstack-ansible-openstack-ansible-aio-ubuntu-xenial http://logs.openstack.org/12/461012/1/check/gate-openstack-ansible-openstack-ansible-aio-ubuntu-xenial/1c1ced8/ : SUCCESS in 1h 17m 16s\n- gate-openstack-ansible-openstack-ansible-ceph-ubuntu-xenial-nv http://logs.openstack.org/12/461012/1/check/gate-openstack-ansible-openstack-ansible-ceph-ubuntu-xenial-nv/252bfd2/ : SUCCESS in 59m 31s (non-voting)\n- gate-openstack-ansible-openstack-ansible-ceph-centos-7-nv http://logs.openstack.org/12/461012/1/check/gate-openstack-ansible-openstack-ansible-ceph-centos-7-nv/97b6c6d/ : FAILURE in 21m 10s (non-voting)\n- gate-openstack-ansible-openstack-ansible-aio-centos-7-nv http://logs.openstack.org/12/461012/1/check/gate-openstack-ansible-openstack-ansible-aio-centos-7-nv/14d3df9/ : FAILURE in 21m 26s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"cb753befa9d6b2209a39f921c081cef2a5781382","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-04-28 13:37:51.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"e025dd1090f91b5bc67505ebea06a1911ad54fe5","date":"2017-04-28 14:54:16.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1},{"id":"b4f16d420decb445af058c0eba00575609064da1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-04-28 14:54:16.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-openstack-ansible-docs-ubuntu-xenial http://docs-draft.openstack.org/12/461012/1/gate/gate-openstack-ansible-docs-ubuntu-xenial/57259a9//doc/build/html/ : SUCCESS in 1m 22s\n- gate-openstack-ansible-linters-ubuntu-xenial http://logs.openstack.org/12/461012/1/gate/gate-openstack-ansible-linters-ubuntu-xenial/4a6b91f/ : SUCCESS in 8m 33s\n- gate-openstack-ansible-openstack-ansible-aio-ubuntu-xenial http://logs.openstack.org/12/461012/1/gate/gate-openstack-ansible-openstack-ansible-aio-ubuntu-xenial/a34b863/ : SUCCESS in 1h 16m 05s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"06269c1884f95f946c44b6a708128de69fb002f2","revisions":{"06269c1884f95f946c44b6a708128de69fb002f2":{"kind":"REWORK","_number":1,"created":"2017-04-28 12:18:02.000000000","uploader":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"ref":"refs/changes/12/461012/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/12/461012/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/12/461012/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/12/461012/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/12/461012/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/12/461012/1"}}},"commit":{"parents":[{"commit":"f5c72e5076382918506abea13cdf04291f24936c","subject":"Bugfix for wait_for_ssh for waiting for connectivity","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/f5c72e5076382918506abea13cdf04291f24936c"}]}],"author":{"name":"Jesse Pretorius","email":"jesse.pretorius@rackspace.co.uk","date":"2017-04-28 12:15:44.000000000","tz":60},"committer":{"name":"Jesse Pretorius","email":"jesse.pretorius@rackspace.co.uk","date":"2017-04-28 12:17:51.000000000","tz":60},"subject":"Use wait_for_connection instead of wait_for to check container","message":"Use wait_for_connection instead of wait_for to check container\n\nAfter creating the container or configuring the bind mounts for\nthe container, we used a task to wait for SSH to respond on the\ncontainer, using the Ansible wait_for module trying to connect to\nport 22.\n\nWhen managing containers in Ansible using other connection plugins\nthat use LXC commands on the physical host to execute modules,\nthere may not be an SSH daemon running within the container. In\nthese situations we can accomplish the same check without SSH by\nusing the wait_for_connection module (new in ansible 2.3) to verify\nthat Ansible has a working execution path into the container, but\nnot necessarily via SSH.\n\nChange-Id: I1399b6727ec8894dd3b9c464f6addbeea1c1f4f9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/06269c1884f95f946c44b6a708128de69fb002f2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/06269c1884f95f946c44b6a708128de69fb002f2"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}