)]}'
{"id":"openstack%2Fdevstack~41799","triplet_id":"openstack%2Fdevstack~master~I8e17ba68093faafe58a98eb780a032368eea38aa","project":"openstack/devstack","branch":"master","topic":"iputil","hashtags":[],"change_id":"I8e17ba68093faafe58a98eb780a032368eea38aa","subject":"Added functions for get a ip on an instance","status":"MERGED","created":"2013-08-13 22:02:18.000000000","updated":"2013-08-16 02:55:11.000000000","submitted":"2013-08-16 02:55:11.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"626028c70e9f2e10e34cfc519641734488def47a","_number":41799,"virtual_id_number":41799,"owner":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":2,"date":"2013-08-16 02:55:11.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"all":[{"value":1,"date":"2013-08-15 19:40:25.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":1,"date":"2013-08-14 18:41:33.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-08-16 01:20:15.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},{"value":1,"date":"2013-08-14 18:40:14.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-08-16 01:20:15.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-08-13 22:03:11.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-08-14 18:40:14.000000000","updated_by":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"reviewer":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"state":"REVIEWER"},{"updated":"2013-08-14 18:41:33.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2013-08-15 19:40:25.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2013-08-16 01:20:15.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-08-16 02:55:11.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"bc9dc5bffcfd42355173a134d578925f6e73c204","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-13 22:53:49.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/99/41799/1/check/gate-tempest-devstack-vm-full/c650b8e : SUCCESS in 39m 04s\n- http://logs.openstack.org/99/41799/1/check/gate-tempest-devstack-vm-cells-full/192e699 : FAILURE in 14m 04s (non-voting)\n- http://logs.openstack.org/99/41799/1/check/gate-tempest-devstack-vm-testr-full/2eff71c : SUCCESS in 24m 59s (non-voting)\n- http://logs.openstack.org/99/41799/1/check/gate-tempest-devstack-vm-postgres-full/a3d5049 : SUCCESS in 51m 25s\n- http://logs.openstack.org/99/41799/1/check/gate-tempest-devstack-vm-neutron/09a4d27 : SUCCESS in 18m 43s\n- http://logs.openstack.org/99/41799/1/check/gate-grenade-devstack-vm/85e10ff : SUCCESS in 21m 58s\n","accounts_in_message":[],"_revision_number":1},{"id":"32020742698d5bfaf0c2c027c11a50cbdb989ce8","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-08-14 00:11:54.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nOverall this looks good. One small suggestion to be a little more defensive inline.","accounts_in_message":[],"_revision_number":1},{"id":"b5bff92f414d467c2a8df0d6f1311a9c37d4c2bf","author":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"date":"2013-08-14 00:25:59.000000000","message":"Patch Set 1: (1 inline comment)\n\nThank you for your review! fixed","accounts_in_message":[],"_revision_number":1},{"id":"11b483c8c0d1fc8ee9b8ea59beafde4659cf5173","author":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"date":"2013-08-14 00:26:06.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"7ced1892221da95b90014e891fb186a3fba78fb9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-14 01:17:37.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/99/41799/2/check/gate-tempest-devstack-vm-full/42e67a3 : SUCCESS in 41m 49s\n- http://logs.openstack.org/99/41799/2/check/gate-tempest-devstack-vm-cells-full/10b16df : FAILURE in 13m 25s (non-voting)\n- http://logs.openstack.org/99/41799/2/check/gate-tempest-devstack-vm-testr-full/1d3be76 : FAILURE in 24m 03s (non-voting)\n- http://logs.openstack.org/99/41799/2/check/gate-tempest-devstack-vm-postgres-full/126f1d7 : SUCCESS in 51m 24s\n- http://logs.openstack.org/99/41799/2/check/gate-tempest-devstack-vm-neutron/70dad14 : SUCCESS in 17m 52s\n- http://logs.openstack.org/99/41799/2/check/gate-grenade-devstack-vm/aaff3a7 : SUCCESS in 21m 58s\n","accounts_in_message":[],"_revision_number":2},{"id":"b2b0a96dc033a319e02913322b1e81c69699e158","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2013-08-14 18:40:14.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"bac9b06e6d8d3523a51a940b0f6e25f93f912bcd","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2013-08-14 18:41:33.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"51fd2463038228021c152f72748fe453c7f5b3d6","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-08-15 19:40:25.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"d7eef9890a2eb71b46c714c7dd31b08110d6a627","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-08-16 01:20:15.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"55702f562257b5399eec4afb3de5c716eb6b8e88","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-16 01:20:38.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"6748f91cc009cc28a4008245ed31c1fe72d5fc76","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-16 02:55:11.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2},{"id":"5d1cee5ec5444e01b1c7425b64890b4fb06752a6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-16 02:55:11.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/99/41799/2/gate/gate-tempest-devstack-vm-full/446620e : SUCCESS in 39m 52s\n- http://logs.openstack.org/99/41799/2/gate/gate-tempest-devstack-vm-testr-full/f385ef4 : SUCCESS in 28m 45s (non-voting)\n- http://logs.openstack.org/99/41799/2/gate/gate-tempest-devstack-vm-postgres-full/3f557b2 : SUCCESS in 41m 33s\n- http://logs.openstack.org/99/41799/2/gate/gate-tempest-devstack-vm-neutron/56fd9cd : SUCCESS in 17m 33s\n- http://logs.openstack.org/99/41799/2/gate/gate-grenade-devstack-vm/7819e78 : SUCCESS in 25m 49s\n","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"6769b166b10272947db77c3f9bfb0d115e8d0a2d","revisions":{"b1a58e1d9cb1a6a4bcf77248e1861dddfa842b56":{"kind":"REWORK","_number":1,"created":"2013-08-13 22:02:18.000000000","uploader":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"ref":"refs/changes/99/41799/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/99/41799/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/99/41799/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/99/41799/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/99/41799/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/99/41799/1"}}},"commit":{"parents":[{"commit":"55181d2cfba5dd6bbbde42bb964782e8c2b8ee50","subject":"Merge \"Pip install fails because of --use-mirrors parameter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/55181d2cfba5dd6bbbde42bb964782e8c2b8ee50"}]}],"author":{"name":"Nachi Ueno","email":"nachi@ntti3.com","date":"2013-08-13 01:18:56.000000000","tz":-420},"committer":{"name":"Nachi Ueno","email":"nachi@ntti3.com","date":"2013-08-13 21:44:48.000000000","tz":-420},"subject":"Added functions for get a ip on an instance","message":"Added functions for get a ip on an instance\n\nThe cause of some gating failure looks like\nbecause of getting ip address on instance.\nHowever current exercise didn\u0027t log the return value.\nIn this commit, we add get_instance_ip function with\nerror hanlding support, and apply it on the execise.\n\nChange-Id: I8e17ba68093faafe58a98eb780a032368eea38aa\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b1a58e1d9cb1a6a4bcf77248e1861dddfa842b56"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b1a58e1d9cb1a6a4bcf77248e1861dddfa842b56"}]},"branch":"refs/heads/master"},"6769b166b10272947db77c3f9bfb0d115e8d0a2d":{"kind":"REWORK","_number":2,"created":"2013-08-14 00:26:06.000000000","uploader":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"ref":"refs/changes/99/41799/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/99/41799/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/99/41799/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/99/41799/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/99/41799/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/99/41799/2"}}},"commit":{"parents":[{"commit":"55181d2cfba5dd6bbbde42bb964782e8c2b8ee50","subject":"Merge \"Pip install fails because of --use-mirrors parameter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/55181d2cfba5dd6bbbde42bb964782e8c2b8ee50"}]}],"author":{"name":"Nachi Ueno","email":"nachi@ntti3.com","date":"2013-08-13 01:18:56.000000000","tz":-420},"committer":{"name":"Nachi Ueno","email":"nachi@ntti3.com","date":"2013-08-14 00:08:47.000000000","tz":-420},"subject":"Added functions for get a ip on an instance","message":"Added functions for get a ip on an instance\n\nThe cause of some gating failure looks like\nbecause of getting ip address on instance.\nHowever current exercise didn\u0027t log the return value.\nIn this commit, we add get_instance_ip function with\nerror hanlding support, and apply it on the execise.\n\nChange-Id: I8e17ba68093faafe58a98eb780a032368eea38aa\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6769b166b10272947db77c3f9bfb0d115e8d0a2d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6769b166b10272947db77c3f9bfb0d115e8d0a2d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}