)]}'
{"id":"openstack%2Fopenstack-ansible-plugins~475598","triplet_id":"openstack%2Fopenstack-ansible-plugins~master~If594914df53efacc6d5bba148f4f46280f5a117d","project":"openstack/openstack-ansible-plugins","branch":"master","topic":"fix-multihost-delegation","hashtags":[],"change_id":"If594914df53efacc6d5bba148f4f46280f5a117d","subject":"Fix delegation to containers between hosts","status":"MERGED","created":"2017-06-19 22:48:58.000000000","updated":"2017-06-20 04:53:25.000000000","submitted":"2017-06-20 04:53:25.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0d13f6525851b53fb63a96401cbc2da0d9eff68b","_number":475598,"virtual_id_number":475598,"owner":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2017-06-20 04:53:25.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":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"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-06-20 01:06:08.000000000","_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"value":2,"date":"2017-06-20 04:51:36.000000000","_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"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":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"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-06-20 04:51:36.000000000","_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"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":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"}],"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":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-06-20 01:06:08.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-06-20 04:51:36.000000000","updated_by":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"reviewer":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"state":"REVIEWER"},{"updated":"2017-06-20 04:53:25.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"6fffae8529903f54c870cc709f53bf34671a761f","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2017-06-19 22:48:58.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"73ea2e5e46d4483c8ab9ded87199a1a9c499a29c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-19 22:52:02.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-openstack-ansible-plugins-docs-ubuntu-xenial http://docs-draft.openstack.org/98/475598/1/check/gate-openstack-ansible-plugins-docs-ubuntu-xenial/d6f315b//doc/build/html/ : SUCCESS in 49s\n- gate-openstack-ansible-plugins-linters-ubuntu-xenial http://logs.openstack.org/98/475598/1/check/gate-openstack-ansible-plugins-linters-ubuntu-xenial/32b50cd/ : SUCCESS in 49s\n- gate-openstack-ansible-plugins-ansible-func-centos-7 http://logs.openstack.org/98/475598/1/check/gate-openstack-ansible-plugins-ansible-func-centos-7/af6a620/ : FAILURE in 1m 25s\n- gate-openstack-ansible-plugins-ansible-func-ubuntu-xenial http://logs.openstack.org/98/475598/1/check/gate-openstack-ansible-plugins-ansible-func-ubuntu-xenial/15a77ba/ : FAILURE in 1m 04s","accounts_in_message":[],"_revision_number":1},{"id":"6c433189cc8fbdcb9c1d152d986da41d5029d4d7","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2017-06-19 23:14:56.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"c3815076b44cf5b52a7ef917ac048b5dd1715ce3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-19 23:18:09.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-openstack-ansible-plugins-docs-ubuntu-xenial http://docs-draft.openstack.org/98/475598/2/check/gate-openstack-ansible-plugins-docs-ubuntu-xenial/7aac748//doc/build/html/ : SUCCESS in 1m 43s\n- gate-openstack-ansible-plugins-linters-ubuntu-xenial http://logs.openstack.org/98/475598/2/check/gate-openstack-ansible-plugins-linters-ubuntu-xenial/631434e/ : SUCCESS in 1m 20s\n- gate-openstack-ansible-plugins-ansible-func-centos-7 http://logs.openstack.org/98/475598/2/check/gate-openstack-ansible-plugins-ansible-func-centos-7/d929cb7/ : SUCCESS in 1m 29s\n- gate-openstack-ansible-plugins-ansible-func-ubuntu-xenial http://logs.openstack.org/98/475598/2/check/gate-openstack-ansible-plugins-ansible-func-ubuntu-xenial/b83e2be/ : SUCCESS in 1m 19s","accounts_in_message":[],"_revision_number":2},{"id":"c271a1f69ebc44b3ef7f0903d01eee0097246722","author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"date":"2017-06-20 01:06:08.000000000","message":"Patch Set 2: Code-Review+2\n\n(1 comment)\n\nGood catch! Reproduced the bug and confirmed this fix on a test env.","accounts_in_message":[],"_revision_number":2},{"id":"1b7c3bf05f31a4c09d50646afcbaff1c5614d72b","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2017-06-20 01:17:32.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"7489190fa977acddfd7a0417edddfe3163814381","author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"date":"2017-06-20 01:24:05.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"9bc08fad7b69c0a841e62e8a51516f530b195f33","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2017-06-20 04:51:36.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"ebd7c318b8adae1af4214184e1a5e0243dace44a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-20 04:51:47.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"1a762f20a13c68b481ed19b1bb16a6e5ecc268b8","date":"2017-06-20 04:53:25.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"0a88a0318c9e9c45bc5530b3346436b77e4e82ff","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-20 04:53:25.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-openstack-ansible-plugins-docs-ubuntu-xenial http://docs-draft.openstack.org/98/475598/2/gate/gate-openstack-ansible-plugins-docs-ubuntu-xenial/cf25dd6//doc/build/html/ : SUCCESS in 1m 16s\n- gate-openstack-ansible-plugins-linters-ubuntu-xenial http://logs.openstack.org/98/475598/2/gate/gate-openstack-ansible-plugins-linters-ubuntu-xenial/789d015/ : SUCCESS in 1m 33s\n- gate-openstack-ansible-plugins-ansible-func-centos-7 http://logs.openstack.org/98/475598/2/gate/gate-openstack-ansible-plugins-ansible-func-centos-7/c8e1b7a/ : SUCCESS in 1m 23s\n- gate-openstack-ansible-plugins-ansible-func-ubuntu-xenial http://logs.openstack.org/98/475598/2/gate/gate-openstack-ansible-plugins-ansible-func-ubuntu-xenial/82ad4d9/ : SUCCESS in 1m 02s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"8b1a07d2d60cb5e0a48fc152f1eee8a3bc051902","revisions":{"223081f93ab66b17d5ec8c1f551e90bd8ab91128":{"kind":"REWORK","_number":1,"created":"2017-06-19 22:48:58.000000000","uploader":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"ref":"refs/changes/98/475598/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-plugins","ref":"refs/changes/98/475598/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/1"}}},"commit":{"parents":[{"commit":"d66fb8f310d1eac7ee8ee28d07ed18ed18ee3313","subject":"Merge \"Python fix to the connection issue\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/d66fb8f310d1eac7ee8ee28d07ed18ed18ee3313"}]}],"author":{"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","date":"2017-06-19 22:38:22.000000000","tz":-420},"committer":{"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","date":"2017-06-19 22:48:16.000000000","tz":-420},"subject":"Fix delegation to containers between hosts","message":"Fix delegation to containers between hosts\n\nTasks which delegate from a container to a container on a separate host\ncurrently fail because the \u0027physical_host_addr\u0027 variable, which those\ntasks use to connect to a physical host, is exclusively set as the\naddress of the originating container\u0027s physical host.\n\nAssign a new host variable, \u0027physical_host_addrs\u0027, which contains a\nmapping of each physical host and its address, so the tasks can now\nlookup of the correct address to use depending on the target container.\n\nChange-Id: If594914df53efacc6d5bba148f4f46280f5a117d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/223081f93ab66b17d5ec8c1f551e90bd8ab91128"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/223081f93ab66b17d5ec8c1f551e90bd8ab91128"}]},"branch":"refs/heads/master"},"8b1a07d2d60cb5e0a48fc152f1eee8a3bc051902":{"kind":"REWORK","_number":2,"created":"2017-06-19 23:14:56.000000000","uploader":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"ref":"refs/changes/98/475598/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-plugins","ref":"refs/changes/98/475598/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-plugins refs/changes/98/475598/2"}}},"commit":{"parents":[{"commit":"d66fb8f310d1eac7ee8ee28d07ed18ed18ee3313","subject":"Merge \"Python fix to the connection issue\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/d66fb8f310d1eac7ee8ee28d07ed18ed18ee3313"}]}],"author":{"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","date":"2017-06-19 22:38:22.000000000","tz":-420},"committer":{"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","date":"2017-06-19 23:14:47.000000000","tz":-420},"subject":"Fix delegation to containers between hosts","message":"Fix delegation to containers between hosts\n\nTasks which delegate from a container to a container on a separate host\ncurrently fail because the \u0027physical_host_addr\u0027 variable, which those\ntasks use to connect to a physical host, is exclusively set as the\naddress of the originating container\u0027s physical host.\n\nAssign a new host variable, \u0027physical_host_addrs\u0027, which contains a\nmapping of each physical host and its address, so the tasks can now\nlookup of the correct address to use depending on the target container.\n\nChange-Id: If594914df53efacc6d5bba148f4f46280f5a117d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/8b1a07d2d60cb5e0a48fc152f1eee8a3bc051902"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-plugins/commit/8b1a07d2d60cb5e0a48fc152f1eee8a3bc051902"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}