)]}'
{"id":"openstack%2Fnova~24921","triplet_id":"openstack%2Fnova~master~Ide35ad5d8e147f6bd21b5840b901363dab0eae38","project":"openstack/nova","branch":"master","topic":"bug/1157408","hashtags":[],"change_id":"Ide35ad5d8e147f6bd21b5840b901363dab0eae38","subject":"Change arguments to volume_detach()","status":"MERGED","created":"2013-03-20 18:50:05.000000000","updated":"2013-03-22 14:51:40.000000000","submitted":"2013-03-22 14:51:37.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":7,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"cc51f4845d28d578058cf1b10237df8a8f5c58bb","_number":24921,"virtual_id_number":24921,"owner":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":2,"date":"2013-03-22 14:51:37.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":1,"date":"2013-03-20 21:06:39.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"}],"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":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"all":[{"value":2,"date":"2013-03-22 14:15:37.000000000","_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-03-20 21:05:24.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":1,"date":"2013-03-20 23:31:44.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":1,"date":"2013-03-21 00:09:17.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":2,"date":"2013-03-20 20:52:09.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":1,"date":"2013-03-20 20:56:11.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},{"value":1,"date":"2013-03-22 11:32:30.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"}],"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":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"all":[{"value":1,"date":"2013-03-22 14:15:37.000000000","_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"date":"2013-03-20 20:49:17.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-03-20 19:34:12.000000000","updated_by":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"reviewer":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"state":"REVIEWER"},{"updated":"2013-03-20 20:49:17.000000000","updated_by":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"reviewer":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"state":"REVIEWER"},{"updated":"2013-03-20 20:56:11.000000000","updated_by":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"reviewer":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"state":"REVIEWER"},{"updated":"2013-03-20 21:05:24.000000000","updated_by":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"reviewer":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"state":"REVIEWER"},{"updated":"2013-03-20 21:06:39.000000000","updated_by":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"reviewer":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2013-03-20 23:31:44.000000000","updated_by":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"reviewer":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"state":"REVIEWER"},{"updated":"2013-03-21 00:09:17.000000000","updated_by":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"reviewer":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"state":"REVIEWER"},{"updated":"2013-03-22 11:32:30.000000000","updated_by":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"reviewer":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"state":"REVIEWER"},{"updated":"2013-03-22 14:15:37.000000000","updated_by":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"reviewer":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"state":"REVIEWER"},{"updated":"2013-03-22 14:51:37.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"d435dc2c249aa7e00f82ce3c18e299ae49c71c83","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 18:51:57.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"c7a389d78c378f7342ab32e410bf33621e87b085","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-03-20 18:52:19.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"5f2f6a1eb9dcc6126b340690c45959f21ed122e3","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 18:56:10.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"1ac2714191275b1ce96e23c4b3def16caeeaebc3","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 18:59:45.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"d93c37138165773532c649228fa97f1ed804ffaa","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 19:04:51.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"803a86c903df58645584445135518cff656ad851","author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"date":"2013-03-20 19:25:20.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n(1 inline comment)\n\nLGTM","accounts_in_message":[],"_revision_number":5},{"id":"3a1776d3aac691d96cfc6a101f24fbbf992877c1","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-03-20 19:27:36.000000000","message":"Patch Set 5: Works for me\n\nSmokeStack Results (patch set 5):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/78984\n\tXenServer 5.6 (F18 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/78985\n","accounts_in_message":[],"_revision_number":5},{"id":"dd7e8586564ceaf557c47cf58eac47867cb0bd67","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 19:30:39.000000000","message":"Patch Set 5: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":5},{"id":"cd1921f95763d8fb43761d0ac7d742551b467a59","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 19:31:17.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"4e5304a975efd18336cc8068773ddf9f353dad0d","author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"date":"2013-03-20 19:34:53.000000000","message":"Patch Set 6: Looks good to me, but someone else must approve\n\nHeh, took me way too long to find that typo even though you pointed it out.","accounts_in_message":[],"_revision_number":6},{"id":"36c03306442eca701d958afa193faaa3febb099f","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-03-20 19:50:01.000000000","message":"Patch Set 6: Works for me\n\nSmokeStack Results (patch set 6):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/78989\n\tXenServer 5.6 (F18 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/78990\n","accounts_in_message":[],"_revision_number":6},{"id":"f73f155a23f846b71e8bbdf21a23312a083baf82","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2013-03-20 19:59:15.000000000","message":"Patch Set 6: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":6},{"id":"1fee8974eea5543acaa32b20e819ca50061c2fab","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-20 20:02:39.000000000","message":"Patch Set 6: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/24921/6/check/gate-nova-pep8/23308 : SUCCESS\n- http://logs.openstack.org/24921/6/check/gate-nova-docs/17647 : SUCCESS\n- http://logs.openstack.org/24921/6/check/gate-nova-python26/19056 : SUCCESS\n- http://logs.openstack.org/24921/6/check/gate-nova-python27/19144 : SUCCESS\n- http://logs.openstack.org/24921/6/check/gate-tempest-devstack-vm-full/9738 : FAILURE\n- http://logs.openstack.org/24921/6/check/gate-tempest-devstack-vm-postgres-full/9453 : FAILURE\n- http://logs.openstack.org/24921/6/check/gate-tempest-devstack-vm-quantum/15155 : FAILURE\n- http://logs.openstack.org/24921/6/check/gate-nova-pylint/9662 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":6},{"id":"492cc0ff836f70d4c11a44f79419921ed6756b11","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 20:38:57.000000000","message":"Patch Set 6: (1 inline comment)\n\nreply to russell inline.","accounts_in_message":[],"_revision_number":6},{"id":"6b249625736a574f7bacb9fe7da87bb48f6950dc","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 20:41:49.000000000","message":"Patch Set 6: (1 inline comment)\n\ncomment inline about devstack failure due to policy check change.. hm.","accounts_in_message":[],"_revision_number":6},{"id":"a26384b0ae9438d2cf7e5ac6099192c0c1be70a7","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 20:43:29.000000000","message":"Patch Set 6: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":6},{"id":"8bb313ad97bb83e6563b4a80647e94843f7da723","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-03-20 20:49:07.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"b28cd4dcae1efb41182f4f3a7b207f738e233f56","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2013-03-20 20:52:09.000000000","message":"Patch Set 7: Looks good to me (core reviewer)\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":7},{"id":"292af552e44dd830397a8e5c1a88ca15fc0a0ead","author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"date":"2013-03-20 20:56:11.000000000","message":"Patch Set 7: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":7},{"id":"38d95ebd8395759860fbf2b6753c409d01076ee1","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-03-20 21:05:24.000000000","message":"Patch Set 7: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":7},{"id":"77a7e24e6a7a7fb5e39cfca7a231dbc32c43ce48","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-03-20 21:06:39.000000000","message":"Patch Set 7: Works for me\n\nSmokeStack Results (patch set 7):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/79003\n\tXenServer 5.6 (F18 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/79004\n","accounts_in_message":[],"_revision_number":7},{"id":"a37e4b29a0e8ea5f8fd391eaa1377bd79ec2188d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-20 21:31:23.000000000","message":"Patch Set 7: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/24921/7/check/gate-nova-pep8/23318 : SUCCESS\n- http://logs.openstack.org/24921/7/check/gate-nova-docs/17657 : SUCCESS\n- http://logs.openstack.org/24921/7/check/gate-nova-python26/19066 : SUCCESS\n- http://logs.openstack.org/24921/7/check/gate-nova-python27/19154 : SUCCESS\n- http://logs.openstack.org/24921/7/check/gate-tempest-devstack-vm-full/9753 : SUCCESS\n- http://logs.openstack.org/24921/7/check/gate-tempest-devstack-vm-postgres-full/9468 : SUCCESS\n- http://logs.openstack.org/24921/7/check/gate-tempest-devstack-vm-quantum/15170 : SUCCESS\n- http://logs.openstack.org/24921/7/check/gate-nova-pylint/9668 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":7},{"id":"afee362fac42f107f5ebb78a77acc9d3847742a6","author":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"date":"2013-03-20 23:31:44.000000000","message":"Patch Set 7: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":7},{"id":"f81b427070d2696c17b5ebe257ad4a0020f21b36","author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"date":"2013-03-21 00:09:17.000000000","message":"Patch Set 7: Looks good to me, but someone else must approve\n\nlgtm.","accounts_in_message":[],"_revision_number":7},{"id":"b51b630971ce239717fcb3dbacec1bbb55c0b31a","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-03-22 11:32:30.000000000","message":"Patch Set 7: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":7},{"id":"c4b195f90ef55f094c7f06a1b9fed93d87a6cc08","author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"date":"2013-03-22 14:15:37.000000000","message":"Patch Set 7: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":7},{"id":"5727ecf31b66874378c60b4f4aea7a6f89f14227","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-22 14:15:45.000000000","message":"Patch Set 7: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":7},{"id":"a672a65b09e1f764929ad765152ce74731d172f3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-22 14:51:37.000000000","message":"Patch Set 7: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/24921/7/gate/gate-nova-docs/17744 : SUCCESS\n- http://logs.openstack.org/24921/7/gate/gate-nova-pep8/23413 : SUCCESS\n- http://logs.openstack.org/24921/7/gate/gate-nova-python26/19153 : SUCCESS\n- http://logs.openstack.org/24921/7/gate/gate-nova-python27/19241 : SUCCESS\n- http://logs.openstack.org/24921/7/gate/gate-tempest-devstack-vm-full/10053 : SUCCESS\n- http://logs.openstack.org/24921/7/gate/gate-tempest-devstack-vm-postgres-full/9768 : SUCCESS\n- http://logs.openstack.org/24921/7/gate/gate-tempest-devstack-vm-quantum/15528 : SUCCESS\n","accounts_in_message":[],"_revision_number":7},{"id":"277a51c04f78c509e8a67f54b95508a6a23f2b24","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-22 14:51:40.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":7}],"current_revision_number":7,"current_revision":"26aa01094a79939320d58f2fe2d5731f169987b1","revisions":{"70707806e27ca18756497713808252e91e119d4a":{"kind":"REWORK","_number":1,"created":"2013-03-20 18:50:05.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/21/24921/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/24921/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/24921/1"}}},"commit":{"parents":[{"commit":"c6b1d4a7c332114f32799c2fd4c346789c45b0f0","subject":"Merge \"Prevent volume-attach/detach from instances in rescue state\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c6b1d4a7c332114f32799c2fd4c346789c45b0f0"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:13:05.000000000","tz":0},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:47:19.000000000","tz":0},"subject":"Change arguments to volume_detach()","message":"Change arguments to volume_detach()\n\nMost compute API calls accept an instance object.  To be compatible with\ncells, volume_detach() should as well.  Also change it to accept the\nvolume object, instead of just a volume ID.  This lets us proxy the\nvolume object to child cells a bit more easily.\n\nThis removes 4 skipTests that we have related to volumes and cells.\n\nFixes bug 1157408\n\nThere was also a check that if instance_get_by_uuid() returned no\ninstance, it VolumeUnattached should be raised.  However, this check\nwould never work as InstanceNotFound would have been raised.  This\npatch also corrects that.\n\nChange-Id: Ide35ad5d8e147f6bd21b5840b901363dab0eae38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/70707806e27ca18756497713808252e91e119d4a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/70707806e27ca18756497713808252e91e119d4a"}]},"branch":"refs/heads/master"},"953b9796253c2d6e7cfe3d3ca227746aa8d5bc49":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-03-20 18:51:57.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/21/24921/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/24921/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/24921/2"}}},"commit":{"parents":[{"commit":"c6b1d4a7c332114f32799c2fd4c346789c45b0f0","subject":"Merge \"Prevent volume-attach/detach from instances in rescue state\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c6b1d4a7c332114f32799c2fd4c346789c45b0f0"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:13:05.000000000","tz":0},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:50:24.000000000","tz":0},"subject":"Change arguments to volume_detach()","message":"Change arguments to volume_detach()\n\nMost compute API calls accept an instance object.  To be compatible with\ncells, volume_detach() should as well.  Also change it to accept the\nvolume object, instead of just a volume ID.  This lets us proxy the\nvolume object to child cells a bit more easily.\n\nThis removes 2 skipTests that we have related to volumes and cells.\n\nFixes bug 1157408\n\nThere was also a check that if instance_get_by_uuid() returned no\ninstance, it VolumeUnattached should be raised.  However, this check\nwould never work as InstanceNotFound would have been raised.  This\npatch also corrects that.\n\nChange-Id: Ide35ad5d8e147f6bd21b5840b901363dab0eae38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/953b9796253c2d6e7cfe3d3ca227746aa8d5bc49"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/953b9796253c2d6e7cfe3d3ca227746aa8d5bc49"}]},"branch":"refs/heads/master"},"1362d8eb9a7c24a86aa66f3207164b3e37ee0aa7":{"kind":"REWORK","_number":3,"created":"2013-03-20 18:56:10.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/21/24921/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/24921/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/24921/3"}}},"commit":{"parents":[{"commit":"c6b1d4a7c332114f32799c2fd4c346789c45b0f0","subject":"Merge \"Prevent volume-attach/detach from instances in rescue state\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c6b1d4a7c332114f32799c2fd4c346789c45b0f0"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:13:05.000000000","tz":0},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:54:41.000000000","tz":0},"subject":"Change arguments to volume_detach()","message":"Change arguments to volume_detach()\n\nMost compute API calls accept an instance object.  To be compatible with\ncells, volume_detach() should as well.  Also change it to accept the\nvolume object, instead of just a volume ID.  This lets us proxy the\nvolume object to child cells a bit more easily.\n\nThis removes 2 skipTests that we have related to volumes and cells.\n\nFixes bug 1157408\n\nThere was also a check that if instance_get_by_uuid() returned no\ninstance, it VolumeUnattached should be raised.  However, this check\nwould never work as InstanceNotFound would have been raised.  This\npatch also corrects that.\n\nChange-Id: Ide35ad5d8e147f6bd21b5840b901363dab0eae38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1362d8eb9a7c24a86aa66f3207164b3e37ee0aa7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1362d8eb9a7c24a86aa66f3207164b3e37ee0aa7"}]},"branch":"refs/heads/master"},"3792258a48cde6d85fa855f6e7da50e72bd9fb39":{"kind":"REWORK","_number":4,"created":"2013-03-20 18:59:45.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/21/24921/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/24921/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/24921/4"}}},"commit":{"parents":[{"commit":"c6b1d4a7c332114f32799c2fd4c346789c45b0f0","subject":"Merge \"Prevent volume-attach/detach from instances in rescue state\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c6b1d4a7c332114f32799c2fd4c346789c45b0f0"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:13:05.000000000","tz":0},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:58:16.000000000","tz":0},"subject":"Change arguments to volume_detach()","message":"Change arguments to volume_detach()\n\nMost compute API calls accept an instance object.  To be compatible with\ncells, volume_detach() should as well.  Also change it to accept the\nvolume object, instead of just a volume ID.  This lets us proxy the\nvolume object to child cells a bit more easily.\n\nThis removes 2 skipTests that we have related to volumes and cells.\n\nFixes bug 1157408\n\nThere was also a check that if instance_get_by_uuid() returned no\ninstance, it VolumeUnattached should be raised.  However, this check\nwould never work as InstanceNotFound would have been raised.  This\npatch also corrects that.\n\nChange-Id: Ide35ad5d8e147f6bd21b5840b901363dab0eae38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3792258a48cde6d85fa855f6e7da50e72bd9fb39"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3792258a48cde6d85fa855f6e7da50e72bd9fb39"}]},"branch":"refs/heads/master"},"35f7381490cf1aa6db1589dd065dfe6d46cea845":{"kind":"REWORK","_number":5,"created":"2013-03-20 19:04:51.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/21/24921/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/24921/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/24921/5"}}},"commit":{"parents":[{"commit":"c6b1d4a7c332114f32799c2fd4c346789c45b0f0","subject":"Merge \"Prevent volume-attach/detach from instances in rescue state\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c6b1d4a7c332114f32799c2fd4c346789c45b0f0"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:13:05.000000000","tz":0},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 19:03:23.000000000","tz":0},"subject":"Change arguments to volume_detach()","message":"Change arguments to volume_detach()\n\nMost compute API calls accept an instance object.  To be compatible with\ncells, volume_detach() should as well.  Also change it to accept the\nvolume object, instead of just a volume ID.  This lets us proxy the\nvolume object to child cells a bit more easily.\n\nThis removes 2 skipTests that we have related to volumes and cells.\n\nFixes bug 1157408\n\nThere was also a check that if instance_get_by_uuid() returned no\ninstance, it VolumeUnattached should be raised.  However, this check\nwould never work as InstanceNotFound would have been raised.  This\npatch also corrects that.\n\nChange-Id: Ide35ad5d8e147f6bd21b5840b901363dab0eae38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/35f7381490cf1aa6db1589dd065dfe6d46cea845"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/35f7381490cf1aa6db1589dd065dfe6d46cea845"}]},"branch":"refs/heads/master"},"f673e63ae61efe715fd6a8a95bc4781a571dfa1f":{"kind":"REWORK","_number":6,"created":"2013-03-20 19:31:17.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/21/24921/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/24921/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/24921/6"}}},"commit":{"parents":[{"commit":"c6b1d4a7c332114f32799c2fd4c346789c45b0f0","subject":"Merge \"Prevent volume-attach/detach from instances in rescue state\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c6b1d4a7c332114f32799c2fd4c346789c45b0f0"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:13:05.000000000","tz":0},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 19:29:47.000000000","tz":0},"subject":"Change arguments to volume_detach()","message":"Change arguments to volume_detach()\n\nMost compute API calls accept an instance object.  To be compatible with\ncells, volume_detach() should as well.  Also change it to accept the\nvolume object, instead of just a volume ID.  This lets us proxy the\nvolume object to child cells a bit more easily.\n\nThis removes 2 skipTests that we have related to volumes and cells.\n\nFixes bug 1157408\n\nThere was also a check that if instance_get_by_uuid() returned no\ninstance, it VolumeUnattached should be raised.  However, this check\nwould never work as InstanceNotFound would have been raised.  This\npatch also corrects that.\n\nChange-Id: Ide35ad5d8e147f6bd21b5840b901363dab0eae38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f673e63ae61efe715fd6a8a95bc4781a571dfa1f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f673e63ae61efe715fd6a8a95bc4781a571dfa1f"}]},"branch":"refs/heads/master"},"26aa01094a79939320d58f2fe2d5731f169987b1":{"kind":"REWORK","_number":7,"created":"2013-03-20 20:49:07.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/21/24921/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/21/24921/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/21/24921/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/21/24921/7"}}},"commit":{"parents":[{"commit":"c6b1d4a7c332114f32799c2fd4c346789c45b0f0","subject":"Merge \"Prevent volume-attach/detach from instances in rescue state\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c6b1d4a7c332114f32799c2fd4c346789c45b0f0"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 18:13:05.000000000","tz":0},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-03-20 20:47:36.000000000","tz":0},"subject":"Change arguments to volume_detach()","message":"Change arguments to volume_detach()\n\nMost compute API calls accept an instance object.  To be compatible with\ncells, volume_detach() should as well.  Also change it to accept the\nvolume object, instead of just a volume ID.  This lets us proxy the\nvolume object to child cells a bit more easily.\n\nThis removes 2 skipTests that we have related to volumes and cells.\n\nFixes bug 1157408\n\nThere was also a check that if instance_get_by_uuid() returned no\ninstance, it VolumeUnattached should be raised.  However, this check\nwould never work as InstanceNotFound would have been raised.  This\npatch also corrects that.\n\nChange-Id: Ide35ad5d8e147f6bd21b5840b901363dab0eae38\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/26aa01094a79939320d58f2fe2d5731f169987b1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/26aa01094a79939320d58f2fe2d5731f169987b1"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
