)]}'
{"id":"openstack%2Fnova~21481","triplet_id":"openstack%2Fnova~master~Iad30b462d49c88fc19babed43a2fb8540b1fad30","project":"openstack/nova","branch":"master","topic":"improve-baremetal-deploy","hashtags":[],"change_id":"Iad30b462d49c88fc19babed43a2fb8540b1fad30","subject":"Add better status to baremetal deployments.","status":"MERGED","created":"2013-02-08 00:52:31.000000000","updated":"2013-02-20 20:03:32.000000000","submitted":"2013-02-20 20:03:29.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":31,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a37a6e9a20a51a6c2524a2217842d405853698b2","_number":21481,"virtual_id_number":21481,"owner":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-02-20 20:03:29.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":1,"date":"2013-02-20 19:10:16.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},{"value":0,"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-02-20 18:27:51.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},{"value":0,"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-02-20 18:27:51.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},{"value":0,"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},{"value":0,"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-02-14 23:09:35.000000000","updated_by":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"reviewer":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"state":"REVIEWER"},{"updated":"2013-02-18 22:28:02.000000000","updated_by":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"reviewer":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"state":"REVIEWER"},{"updated":"2013-02-18 22:28:03.000000000","updated_by":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"reviewer":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"state":"REVIEWER"},{"updated":"2013-02-18 23:18:28.000000000","updated_by":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"reviewer":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"state":"REVIEWER"},{"updated":"2013-02-19 21:09:37.000000000","updated_by":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"reviewer":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"state":"REVIEWER"},{"updated":"2013-02-19 22:28:21.000000000","updated_by":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"reviewer":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"state":"REVIEWER"},{"updated":"2013-02-19 22:51:25.000000000","updated_by":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"reviewer":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"state":"REVIEWER"},{"updated":"2013-02-19 23:51:18.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-02-20 14:43:54.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"},{"updated":"2013-02-20 18:27:51.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-02-20 19:10:16.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-02-20 20:03:29.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"b1dbfc4828589bd661c003fc303a8ebbd09f0dff","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-08 01:51:00.000000000","message":"Patch Set 1: Works for me\n\nSmokeStack Results (patch set 1):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/70388\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/70389\n","accounts_in_message":[],"_revision_number":1},{"id":"6ba88e3fce09d4a8094e3c2f8a79401cceb86e78","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-08 02:22:00.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/1/check/gate-nova-docs/14433 : SUCCESS\n- http://logs.openstack.org/21481/1/check/gate-nova-pep8/19737 : SUCCESS\n- http://logs.openstack.org/21481/1/check/gate-nova-python26/15881 : SUCCESS\n- http://logs.openstack.org/21481/1/check/gate-nova-python27/15940 : SUCCESS\n- http://logs.openstack.org/21481/1/check/gate-tempest-devstack-vm-full/1576 : SUCCESS\n- http://logs.openstack.org/21481/1/check/gate-tempest-devstack-vm-postgres-full/1302 : SUCCESS\n- http://logs.openstack.org/21481/1/check/gate-tempest-devstack-vm-quantum/5618 : SUCCESS\n- http://logs.openstack.org/21481/1/check/gate-nova-pylint/7844 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"d5aab4115ef83fc2cebb8e2de97323c06b63e26c","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-08 06:38:04.000000000","message":"Patch Set 1:\n\nfixing pylint","accounts_in_message":[],"_revision_number":1},{"id":"cdcd7d2b67c647f4cc1feec33c6f81bf0c802513","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-08 06:38:42.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"23c0c7e19a3ad147499f92d378eb8b10987c07cb","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-08 06:40:27.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"ba4369744ad55764c92e206469e242da230400ad","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-02-08 06:40:46.000000000","message":"Patch Set 3:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":3},{"id":"cf479d729bd255c6aa3c902c156d8697db1c89f4","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-08 07:27:36.000000000","message":"Patch Set 3: Works for me\n\nSmokeStack Results (patch set 3):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/70494\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/70495\n","accounts_in_message":[],"_revision_number":3},{"id":"a2dd5e10b6dcb59d705509dccc3ac01a4e3b6c58","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-08 07:49:31.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/3/check/gate-nova-docs/14451 : SUCCESS\n- http://logs.openstack.org/21481/3/check/gate-nova-pep8/19755 : SUCCESS\n- http://logs.openstack.org/21481/3/check/gate-nova-python26/15899 : SUCCESS\n- http://logs.openstack.org/21481/3/check/gate-nova-python27/15958 : SUCCESS\n- http://logs.openstack.org/21481/3/check/gate-tempest-devstack-vm-full/1612 : SUCCESS\n- http://logs.openstack.org/21481/3/check/gate-tempest-devstack-vm-postgres-full/1338 : SUCCESS\n- http://logs.openstack.org/21481/3/check/gate-tempest-devstack-vm-quantum/5663 : SUCCESS\n- http://logs.openstack.org/21481/3/check/gate-nova-pylint/7862 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"71f81653867e7e310e097f6446f5818289bf518e","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-09 09:41:21.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"e85053a13609fd2336a3b2def470a33144154fcd","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-09 11:05:14.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/4/check/gate-nova-docs/14518 : SUCCESS\n- http://logs.openstack.org/21481/4/check/gate-nova-pep8/19822 : SUCCESS\n- http://logs.openstack.org/21481/4/check/gate-nova-python26/15966 : SUCCESS\n- http://logs.openstack.org/21481/4/check/gate-nova-python27/16025 : SUCCESS\n- http://logs.openstack.org/21481/4/check/gate-tempest-devstack-vm-full/1802 : SUCCESS\n- http://logs.openstack.org/21481/4/check/gate-tempest-devstack-vm-postgres-full/1528 : SUCCESS\n- http://logs.openstack.org/21481/4/check/gate-tempest-devstack-vm-quantum/5886 : SUCCESS\n- http://logs.openstack.org/21481/4/check/gate-nova-pylint/7907 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"1fd1ba57ae9b404ec6ce1932374d137399adcc45","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-09 16:32:56.000000000","message":"Patch Set 4: (3 inline comments)\n\nCouple of nitpicks inline.  Otherwise LGTM.","accounts_in_message":[],"_revision_number":4},{"id":"975420d1775d8e975ac22d4ed0b9d49970d075d3","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-10 17:04:25.000000000","message":"Patch Set 4: (3 inline comments)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"74cb3b129c1cea0763553c49ab02ed7bf87f294c","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-10 17:15:13.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"a94c3a52a63d851a019f38720c6ee53a2f3b18be","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-10 17:20:21.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":5},{"id":"9b25a3625fc9d8d4f74c06333e5f257146a41cb6","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-10 17:35:59.000000000","message":"Patch Set 5: Works for me\n\nSmokeStack Results (patch set 5):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/70979\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/70980\n","accounts_in_message":[],"_revision_number":5},{"id":"7d0c9930b7cb4cc87185eef4c7110bb9c7a1e8d8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-10 18:19:42.000000000","message":"Patch Set 5: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/5/check/gate-nova-docs/14567 : SUCCESS\n- http://logs.openstack.org/21481/5/check/gate-nova-pep8/19871 : SUCCESS\n- http://logs.openstack.org/21481/5/check/gate-nova-python26/16015 : SUCCESS\n- http://logs.openstack.org/21481/5/check/gate-nova-python27/16074 : SUCCESS\n- http://logs.openstack.org/21481/5/check/gate-tempest-devstack-vm-full/1899 : SUCCESS\n- http://logs.openstack.org/21481/5/check/gate-tempest-devstack-vm-postgres-full/1625 : SUCCESS\n- http://logs.openstack.org/21481/5/check/gate-tempest-devstack-vm-quantum/5998 : SUCCESS\n- http://logs.openstack.org/21481/5/check/gate-nova-pylint/7945 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":5},{"id":"d79e145aab161c6d4e73c50ea60d8399167b088a","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-12 00:18:51.000000000","message":"Patch Set 5: Work In Progress\n\nlooong conversation with lifeless. Going to retool this somewhat significantly now ...","accounts_in_message":[],"_revision_number":5},{"id":"43ae486d04202814a3fdd2eda1235838217b7f6a","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-13 02:42:34.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"b260f3fd22b491214272d28009b1eeca6e399a62","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-13 02:48:46.000000000","message":"Patch Set 6: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/6/check/gate-nova-pep8/20095 : FAILURE\n- gate-nova-docs : SKIPPED\n- gate-nova-python26 : SKIPPED\n- gate-nova-python27 : SKIPPED\n- gate-tempest-devstack-vm-full : SKIPPED\n- gate-tempest-devstack-vm-postgres-full : SKIPPED\n- gate-tempest-devstack-vm-quantum : SKIPPED\n- gate-nova-pylint : SKIPPED (non-voting)\n","accounts_in_message":[],"_revision_number":6},{"id":"8d1125d906849746fde29c50f542df2899324a44","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-13 03:05:22.000000000","message":"Patch Set 6: Works for me\n\nSmokeStack Results (patch set 6):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/71586\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/71587\n","accounts_in_message":[],"_revision_number":6},{"id":"b20128e37cf9814a56184b7c12f6e00d08b5eab9","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-13 04:55:11.000000000","message":"Patch Set 6: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":6},{"id":"3379fbdd55d5d1c92938da69eda427fd8c3d18c8","author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"date":"2013-02-13 05:10:57.000000000","message":"Patch Set 6: I would prefer that you didn\u0027t merge this\n\n(3 inline comments)\n\nMinor things inline.","accounts_in_message":[],"_revision_number":6},{"id":"3900343ecea14bea454c6b132d7dedb898aefc88","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-13 05:27:20.000000000","message":"Patch Set 6: (3 inline comments)\n\n","accounts_in_message":[],"_revision_number":6},{"id":"0b1c2168afec122a4cf80dbb19533ceb818bd36c","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-13 05:31:33.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"f7b88813fa75e68d243475f385913f23851aaec1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-13 05:42:10.000000000","message":"Patch Set 7: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/7/check/gate-nova-pep8/20119 : FAILURE\n- gate-nova-docs : SKIPPED\n- gate-nova-python26 : SKIPPED\n- gate-nova-python27 : SKIPPED\n- gate-tempest-devstack-vm-full : SKIPPED\n- gate-tempest-devstack-vm-postgres-full : SKIPPED\n- gate-tempest-devstack-vm-quantum : SKIPPED\n- gate-nova-pylint : SKIPPED (non-voting)\n","accounts_in_message":[],"_revision_number":7},{"id":"eb1c00fe0d08114ff7d97638536559c8a256a483","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-13 05:51:59.000000000","message":"Patch Set 7:\n\nwoops... that\u0027s what I get for fixing a one-liner and forgetting to check pep again ...","accounts_in_message":[],"_revision_number":7},{"id":"cbb0bd5117d5bcfada3cf96d36b2d1fb516e763e","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-13 05:55:08.000000000","message":"Patch Set 7: Works for me\n\nSmokeStack Results (patch set 7):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/71625\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/71626\n","accounts_in_message":[],"_revision_number":7},{"id":"5ad067b086eabd980c07730d042c40296c6a6d90","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-13 05:57:39.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"9c54d5188d48a6e0aed091d860159f2e71be3218","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-13 06:20:22.000000000","message":"Patch Set 8: Works for me\n\nSmokeStack Results (patch set 8):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/71635\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/71636\n","accounts_in_message":[],"_revision_number":8},{"id":"e4ebfb0b52f2c3a84c02b58a5630585bd6f346cd","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-13 07:06:52.000000000","message":"Patch Set 8: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/8/check/gate-nova-pep8/20124 : SUCCESS\n- http://logs.openstack.org/21481/8/check/gate-nova-docs/14796 : SUCCESS\n- http://logs.openstack.org/21481/8/check/gate-nova-python26/16243 : SUCCESS\n- http://logs.openstack.org/21481/8/check/gate-nova-python27/16302 : SUCCESS\n- http://logs.openstack.org/21481/8/check/gate-tempest-devstack-vm-full/2467 : SUCCESS\n- http://logs.openstack.org/21481/8/check/gate-tempest-devstack-vm-postgres-full/2194 : SUCCESS\n- http://logs.openstack.org/21481/8/check/gate-tempest-devstack-vm-quantum/6656 : SUCCESS\n- http://logs.openstack.org/21481/8/check/gate-nova-pylint/8110 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":8},{"id":"9dc18f83c48ea4dc22baaca6c4b1235dddf87d72","author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"date":"2013-02-13 07:28:52.000000000","message":"Patch Set 8: I would prefer that you didn\u0027t merge this\n\npylint says we have to import DateTime and Boolean in 002_drop_bm_deployments.py. The others LGTM.","accounts_in_message":[],"_revision_number":8},{"id":"28fed55f7fadd7777e06029fd0e7bdf65754ccbe","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-13 14:45:49.000000000","message":"Patch Set 8:\n\nAre we doing something like test_migrations.py for the baremetal migrations as well?","accounts_in_message":[],"_revision_number":8},{"id":"246a59611c22d4df66901dbee1bc233a65314442","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-13 19:41:31.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"8de8ceeb6244afc49ba79d0f136162e0190bd891","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-13 19:47:27.000000000","message":"Patch Set 9: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/9/check/gate-nova-pep8/20171 : FAILURE\n- gate-nova-docs : SKIPPED\n- gate-nova-python26 : SKIPPED\n- gate-nova-python27 : SKIPPED\n- gate-tempest-devstack-vm-full : SKIPPED\n- gate-tempest-devstack-vm-postgres-full : SKIPPED\n- gate-tempest-devstack-vm-quantum : SKIPPED\n- gate-nova-pylint : SKIPPED (non-voting)\n","accounts_in_message":[],"_revision_number":9},{"id":"94a637393949fd5fd9adec04aa9832347fcbe8c6","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-13 20:07:00.000000000","message":"Patch Set 9: Works for me\n\nSmokeStack Results (patch set 9):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/71793\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/71794\n","accounts_in_message":[],"_revision_number":9},{"id":"2ea9d0dab13501bc1d693381c1f3e523d660d179","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-13 21:09:28.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"d34c3bf75438fdfbcecba0759b1e335a7da1fd2d","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-13 21:14:27.000000000","message":"Patch Set 10: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":10},{"id":"67ec68d44524bf13d6e20cb9e5fc4965066b4ec3","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-13 21:48:06.000000000","message":"Patch Set 10: Works for me\n\nSmokeStack Results (patch set 10):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/71837\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/71838\n","accounts_in_message":[],"_revision_number":10},{"id":"6c03f70b9536a2c6292bd93939f799b1a6ef4147","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-13 23:16:36.000000000","message":"Patch Set 10: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/10/check/gate-nova-pep8/20183 : SUCCESS\n- http://logs.openstack.org/21481/10/check/gate-nova-docs/14851 : SUCCESS\n- http://logs.openstack.org/21481/10/check/gate-nova-python26/16294 : FAILURE\n- http://logs.openstack.org/21481/10/check/gate-nova-python27/16357 : FAILURE\n- http://logs.openstack.org/21481/10/check/gate-tempest-devstack-vm-full/2612 : SUCCESS\n- http://logs.openstack.org/21481/10/check/gate-tempest-devstack-vm-postgres-full/2339 : SUCCESS\n- http://logs.openstack.org/21481/10/check/gate-tempest-devstack-vm-quantum/6828 : SUCCESS\n- http://logs.openstack.org/21481/10/check/gate-nova-pylint/8149 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":10},{"id":"858ddbdc2c90c533ef04ffb084af929a387f79d8","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-14 03:15:35.000000000","message":"Uploaded patch set 11.","accounts_in_message":[],"_revision_number":11},{"id":"dad612d5f7503c36b988043add33a3f1cc16f537","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-14 03:21:43.000000000","message":"Patch Set 11: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/11/check/gate-nova-pep8/20262 : FAILURE\n- gate-nova-docs : SKIPPED\n- gate-nova-python26 : SKIPPED\n- gate-nova-python27 : SKIPPED\n- gate-tempest-devstack-vm-full : SKIPPED\n- gate-tempest-devstack-vm-postgres-full : SKIPPED\n- gate-tempest-devstack-vm-quantum : SKIPPED\n- gate-nova-pylint : SKIPPED (non-voting)\n","accounts_in_message":[],"_revision_number":11},{"id":"c2a29b9117cd798ffdce9e8465f698f40a1f5323","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-02-14 03:30:43.000000000","message":"Patch Set 10: (3 inline comments)\n\nMinor stuff.","accounts_in_message":[],"_revision_number":10},{"id":"201840ae9c51611581c26e824cd13a7cd3e21f49","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-02-14 03:31:13.000000000","message":"Patch Set 11: I would prefer that you didn\u0027t merge this\n\nI had feedback in rev 10, but we raced.","accounts_in_message":[],"_revision_number":11},{"id":"dae48218c66a51c9c57099e82cb4bcc32ed67aba","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-14 03:40:15.000000000","message":"Patch Set 11: Works for me\n\nSmokeStack Results (patch set 11):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/71952\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/71953\n","accounts_in_message":[],"_revision_number":11},{"id":"93632ad1ef04af2f83f3da199f250725426541fc","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-14 04:01:13.000000000","message":"Patch Set 10: (3 inline comments)\n\n","accounts_in_message":[],"_revision_number":10},{"id":"9c3bc0402ba6e50563c8d02cbf0c8e6dd941f19a","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-14 14:40:44.000000000","message":"Uploaded patch set 12.","accounts_in_message":[],"_revision_number":12},{"id":"17a7fa673a3b15e5437ea476118d5fceaed6513c","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-14 15:06:17.000000000","message":"Patch Set 12: Works for me\n\nSmokeStack Results (patch set 12):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/72044\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/72045\n","accounts_in_message":[],"_revision_number":12},{"id":"52868dc06872091cbae17a8b6beb16705c87b97b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-14 16:02:20.000000000","message":"Patch Set 12: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/12/check/gate-nova-pep8/20287 : SUCCESS\n- http://logs.openstack.org/21481/12/check/gate-nova-docs/14934 : SUCCESS\n- http://logs.openstack.org/21481/12/check/gate-nova-python26/16376 : SUCCESS\n- http://logs.openstack.org/21481/12/check/gate-nova-python27/16439 : SUCCESS\n- http://logs.openstack.org/21481/12/check/gate-tempest-devstack-vm-full/2788 : SUCCESS\n- http://logs.openstack.org/21481/12/check/gate-tempest-devstack-vm-postgres-full/2514 : SUCCESS\n- http://logs.openstack.org/21481/12/check/gate-tempest-devstack-vm-quantum/7031 : SUCCESS\n- http://logs.openstack.org/21481/12/check/gate-nova-pylint/8186 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":12},{"id":"0b6cfc9c7e9e50efcf940a71b83fe3ea3c60ddb0","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-14 16:34:19.000000000","message":"Patch Set 12: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":12},{"id":"26fba4e4844cd9bfaf6ca091d990bd34cd790898","author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"date":"2013-02-14 23:09:35.000000000","message":"Patch Set 12: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":12},{"id":"48bd69c5c248970095e616e73180353e41dd2638","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-15 00:43:25.000000000","message":"Patch Set 12:\n\n@Boris, can\u0027t make this a subclass without rewriting a very large chunk of the BaseMigrationTestCase. It references class variables, not class members, pretty heavily. Also, I do not want to inherit all the _pre_* and _check_* methods from the parent class... trying to subclass would mean making a virtual base class that both test classes derive from. Maybe something worth doing in Havana ;)","accounts_in_message":[],"_revision_number":12},{"id":"98a1abce5eb33ca035cdd5823f03ad9c04bdd56f","author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"date":"2013-02-15 04:40:52.000000000","message":"Patch Set 12:\n\nHmm probably you haven\u0027t seen but I splited it already. (so there is TestMigrationBaseCase(abstract class) and TestMigrations\nhttps://review.openstack.org/#/c/20935/\n\nThere is no prerun in TestMigrationBaseCase (only work with engines and config loading and parsing).","accounts_in_message":[],"_revision_number":12},{"id":"1a39e96a34daf08a004da208742d7533f9ba15ec","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-15 18:09:00.000000000","message":"Patch Set 12:\n\n@Boris,\n\nI just looked at your changes. It\u0027s exactly as I said -- your derived class is no different and not reusable. Since baremetal has a different database, different migration and config file, etc, I would still need to duplicate the whole file.\n\nA class derived from your BaseMigrationTestCase will need to not only override the class variables MIGRATE_FILE, etc, it will also need to override setUp; setUp refers to BaseMigrationTestCase many times rather than self.\n\nThe same is true for TestMigrations._walk_versions; this refers to its class variables directly, rather than via self, many times.\n\nAlso, my tests need to\n\n  import nova.virt.baremetal.db.migration as migration\n\nwhereas the existing tests import \"migration\" as\n\n  import nova.db.migration as migration\n\nso again, that reference would need to change all over the code... and these are just a few examples.","accounts_in_message":[],"_revision_number":12},{"id":"6ddad6588acb5cf6c5cd4031459210cb003118eb","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-15 18:10:52.000000000","message":"Patch Set 12:\n\nGoing to post a minor revision based on Mikal\u0027s comments on other patches -- changing (c) header to 2013.","accounts_in_message":[],"_revision_number":12},{"id":"7becea18bd182a042923dfc1fcae410d118d199e","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-15 18:16:53.000000000","message":"Uploaded patch set 13.","accounts_in_message":[],"_revision_number":13},{"id":"7bc6b8238f740ed38a2c11550cd4eb17c197b71d","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-15 18:40:50.000000000","message":"Patch Set 13: Works for me\n\nSmokeStack Results (patch set 13):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/72433\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/72434\n","accounts_in_message":[],"_revision_number":13},{"id":"f5b494b0daa435f77741751762e982ab1d17fc3d","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-15 18:42:35.000000000","message":"Patch Set 13: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":13},{"id":"cdbe0c5584f55f73ca46a5f8b976364d9931c98f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-15 19:09:25.000000000","message":"Patch Set 13: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/13/check/gate-nova-pep8/20396 : SUCCESS\n- http://logs.openstack.org/21481/13/check/gate-nova-docs/15033 : SUCCESS\n- http://logs.openstack.org/21481/13/check/gate-nova-python26/16474 : SUCCESS\n- http://logs.openstack.org/21481/13/check/gate-nova-python27/16538 : SUCCESS\n- http://logs.openstack.org/21481/13/check/gate-tempest-devstack-vm-full/3082 : SUCCESS\n- http://logs.openstack.org/21481/13/check/gate-tempest-devstack-vm-postgres-full/2807 : SUCCESS\n- http://logs.openstack.org/21481/13/check/gate-tempest-devstack-vm-quantum/7372 : SUCCESS\n- http://logs.openstack.org/21481/13/check/gate-nova-pylint/8261 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":13},{"id":"30e60e3af211c5dd05241808ea79432481afd8c5","author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"date":"2013-02-18 04:58:06.000000000","message":"Patch Set 13: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":13},{"id":"a7bf9da8c918f026d3c8bd73a15e98fa8f849574","author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"date":"2013-02-18 05:50:24.000000000","message":"Patch Set 13: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":13},{"id":"e2c12ca8d2687a7948110dca02b983e6c7e402cd","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-02-18 21:18:13.000000000","message":"Patch Set 13: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":13},{"id":"fd3f7bbbe4ab5b084e1ba3d3522184cc8b27d63c","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-02-18 21:20:28.000000000","message":"Patch Set 13: Looks good to me, but someone else must approve\n\n\\o/","accounts_in_message":[],"_revision_number":13},{"id":"45cddf804612dac3ddd3dd9cda5182ac365e8c20","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-02-18 21:37:47.000000000","message":"Patch Set 13: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":13},{"id":"c88912f88889f15b857d58e147768be3913d52d9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-18 21:39:06.000000000","message":"Patch Set 13: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":13},{"id":"6072da246be414533361471356bd9e70f9f82b54","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-18 22:27:30.000000000","message":"Uploaded patch set 14.","accounts_in_message":[],"_revision_number":14},{"id":"1c31e2951186cbdf7961324108690445265b0aff","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-02-18 22:28:02.000000000","message":"Patch Set 14: Looks good to me (core reviewer)\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":14},{"id":"364f5a4995350e402d6ba2fb943354895f83cd40","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-02-18 22:28:02.000000000","message":"Patch Set 14: Looks good to me (core reviewer)\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":14},{"id":"d83f4d9e2a764f0da6c97bed45573dd4969d1ce7","author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"date":"2013-02-18 22:28:02.000000000","message":"Patch Set 14: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":14},{"id":"32dbd1f6de03cd7b4f1abe854e9ce7c884e4627e","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-02-18 22:28:03.000000000","message":"Patch Set 14: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":14},{"id":"110bfa4b19a2c42f7cbd02c073b31ccd4529bc06","author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"date":"2013-02-18 22:28:03.000000000","message":"Patch Set 14: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":14},{"id":"45fcb9fe76b18d41f4086796b70010bdac8097ef","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-18 22:28:03.000000000","message":"Patch Set 14: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":14},{"id":"d5dc73ce3b02ffc519e6a7dfaf83bc9104fa274b","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-18 23:18:28.000000000","message":"Patch Set 14: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nSurely we can avoid a complete copy of test_migrations.py?","accounts_in_message":[],"_revision_number":14},{"id":"4590fa3c37a67987e064fb0ade6fc992cee36213","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-18 23:23:47.000000000","message":"Patch Set 14:\n\nOk, I see the past discussion about the copy and paste ... still makes no sense to me, but I\u0027ll look more closely\n\n\"let\u0027s sort this out in Havana\" doesn\u0027t really cut it for me ... because we rarely actually go back and clean messes like this up\n\n:(","accounts_in_message":[],"_revision_number":14},{"id":"3cf01a69c33040197a3ae9382e1e81b88853a4f6","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-18 23:32:05.000000000","message":"Patch Set 14:\n\nLooked closer and still think sharing code is perfectly doable\n\nSure you\u0027ll have to refactor test_migrations.py a little to allow that, but it\u0027s not a big deal\n\ne.g. you mention the class name being referenced in a bunch of places ... well why not just do:\n\n -        migration_api.version_control(engine, TestMigrations.REPOSITORY,\n -        migration_api.version_control(engine, self.REPOSITORY,\n\n?","accounts_in_message":[],"_revision_number":14},{"id":"8eb9b8660718cda694af6e577243731bcc34ae97","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2013-02-18 23:54:30.000000000","message":"Patch Set 14:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":14},{"id":"211a9d7389e1b1e78b7b3b4f13316263929f0ed8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-19 01:23:59.000000000","message":"Patch Set 14: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/14/check/gate-nova-pep8/20657 : SUCCESS\n- http://logs.openstack.org/21481/14/check/gate-nova-docs/15267 : SUCCESS\n- http://logs.openstack.org/21481/14/check/gate-nova-python26/16708 : SUCCESS\n- http://logs.openstack.org/21481/14/check/gate-nova-python27/16771 : SUCCESS\n- http://logs.openstack.org/21481/14/check/gate-tempest-devstack-vm-full/3752 : SUCCESS\n- http://logs.openstack.org/21481/14/check/gate-tempest-devstack-vm-postgres-full/3473 : SUCCESS\n- http://logs.openstack.org/21481/14/check/gate-tempest-devstack-vm-quantum/8168 : SUCCESS\n- http://logs.openstack.org/21481/14/check/gate-nova-pylint/8401 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":14},{"id":"9038277eb8477150b5f6bbd2badcd848dc1df2d0","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 19:52:47.000000000","message":"Patch Set 14:\n\nTook the last 4 hours, but I think this refactoring will work. Going to push it up now, and then start refactoring all the other patches on top of it. \n\n@Mark, @Vish, if it looks like this is going to prevent these patches from landing in G3, please let me know how to avoid that.\n\nThanks,\nDeva","accounts_in_message":[],"_revision_number":14},{"id":"128a1f357d82e8f46d51211d237a898ec71516ca","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 19:59:05.000000000","message":"Uploaded patch set 15.","accounts_in_message":[],"_revision_number":15},{"id":"12418b0196896d2e19b6d51364fd7e312a2fb52a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-19 19:59:19.000000000","message":"Patch Set 15: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\nThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.","accounts_in_message":[],"_revision_number":15},{"id":"e5da3467d0d56f71f6eff0db2fee14eb1f9dfed6","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 20:42:34.000000000","message":"Uploaded patch set 16.","accounts_in_message":[],"_revision_number":16},{"id":"93c6b8843955bc006ae1446114dcd1156f94ba5d","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-19 20:48:49.000000000","message":"Patch Set 16: (3 inline comments)\n\nA few nits inline, though they might be inherited rather than something you did.","accounts_in_message":[],"_revision_number":16},{"id":"63bf763321878c983a3ab7c7c6f0378c2f316de3","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 20:54:40.000000000","message":"Patch Set 16: (2 inline comments)\n\nfixing the _dict methods","accounts_in_message":[],"_revision_number":16},{"id":"c179eb85d426d26f8357af92ab22fba7c42499e6","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 20:59:17.000000000","message":"Uploaded patch set 17.","accounts_in_message":[],"_revision_number":17},{"id":"77fb38f5dc85980ec98af100214b9ac3c439020e","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-19 21:01:16.000000000","message":"Patch Set 17: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":17},{"id":"3ccc95fcceafa35750b1cf4c1f5b3b7cbcc81838","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-02-19 21:09:37.000000000","message":"Patch Set 17: Looks good to me, but someone else must approve\n\nThanks.","accounts_in_message":[],"_revision_number":17},{"id":"d86c8fe5d2ef829a40c6b29dbe4c434345788a37","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-02-19 21:22:52.000000000","message":"Patch Set 17: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":17},{"id":"9c1fd9d70a33926bdc32f52b46c65225e922e7f2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-19 22:21:49.000000000","message":"Patch Set 17: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/17/check/gate-nova-pep8/20914 : SUCCESS\n- http://logs.openstack.org/21481/17/check/gate-nova-docs/15499 : SUCCESS\n- http://logs.openstack.org/21481/17/check/gate-nova-python26/16941 : SUCCESS\n- http://logs.openstack.org/21481/17/check/gate-nova-python27/17003 : SUCCESS\n- http://logs.openstack.org/21481/17/check/gate-tempest-devstack-vm-full/4212 : SUCCESS\n- http://logs.openstack.org/21481/17/check/gate-tempest-devstack-vm-postgres-full/3931 : SUCCESS\n- http://logs.openstack.org/21481/17/check/gate-tempest-devstack-vm-quantum/8700 : SUCCESS\n- http://logs.openstack.org/21481/17/check/gate-nova-pylint/8485 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":17},{"id":"8c954399436ebaa69aa7a571c5c5012b61c84070","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2013-02-19 22:28:21.000000000","message":"Patch Set 17: Looks good to me (core reviewer); Approved\n\n(2 inline comments)\n\n","accounts_in_message":[],"_revision_number":17},{"id":"cbd48c8e2ad323189088bc5bc1c1007139b973f8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-19 22:28:53.000000000","message":"Patch Set 17: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":17},{"id":"706beab9d2ec18775f0dee380aa533e0ce7141ba","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-02-19 22:40:51.000000000","message":"Patch Set 17: Do not merge\n\nThis is going to fail the gate because the tests that would have failed on it were getting hidden by the XMLMatcher breaks testr issues earlier. Don\u0027t let it land.","accounts_in_message":[],"_revision_number":17},{"id":"bb06828bdc792c47b7252ee29511bdc65f1e10f3","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 22:42:23.000000000","message":"Uploaded patch set 18.","accounts_in_message":[],"_revision_number":18},{"id":"5d48337c3a983d38db003635e377289b23dddfff","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 22:44:24.000000000","message":"Patch Set 17: (1 inline comment)\n\nthere was a bad typo in rev 17 which somehow passed tests, but shouldn\u0027t have.... it\u0027s fixed in rev 18.","accounts_in_message":[],"_revision_number":17},{"id":"4b19b1d277f5ec52295ffdb2612303c424e7ec22","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-19 22:51:25.000000000","message":"Patch Set 18: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n1 nit inline that got missed before.","accounts_in_message":[],"_revision_number":18},{"id":"3ffb0329d8ecef67f9aac4dfb15cc7bc4306abac","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 23:10:37.000000000","message":"Patch Set 18: (1 inline comment)\n\n@dripton, thanks. done now :)","accounts_in_message":[],"_revision_number":18},{"id":"7ffb50a8ba70ef9f6a4091481bd516904983e650","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-19 23:10:57.000000000","message":"Uploaded patch set 19.","accounts_in_message":[],"_revision_number":19},{"id":"f5ad2b63e3b8273ce7aeb40da258519fdb527167","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-02-19 23:51:18.000000000","message":"Patch Set 19: No score\n\n","accounts_in_message":[],"_revision_number":19},{"id":"d32a0cc736315cb36499b9a1a0afe4cc4e7294fe","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 00:41:38.000000000","message":"Patch Set 19: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/19/check/gate-nova-pep8/20964 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-docs/15540 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-python26/16984 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-python27/17044 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-full/4280 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-postgres-full/3999 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-quantum/8775 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-pylint/8520 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":19},{"id":"494d633c48e4b56790152033befffcde3ed2612a","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-20 01:23:09.000000000","message":"Patch Set 19:\n\nThis is not actually being tested properly right now, due to bug https://bugs.launchpad.net/openstack-ci/+bug/1130451. It appears to be passing and getting +1 by Jenkins only because the test_mysql_opportunistically test is not being run (due to the XMLmatcher bug)...","accounts_in_message":[],"_revision_number":19},{"id":"22d9835a811be39932115b103ddbbc4f8a77b971","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-20 01:51:52.000000000","message":"Patch Set 19:\n\nrecheck bug 1130451","accounts_in_message":[],"_revision_number":19},{"id":"a31cd0b8677851119e95baf944d230fedbc186a9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 03:03:01.000000000","message":"Patch Set 19: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/19/check/gate-nova-pep8/21015 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-docs/15598 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-python26/17043 : FAILURE\n- http://logs.openstack.org/21481/19/check/gate-nova-python27/17098 : FAILURE\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-full/4351 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-postgres-full/4071 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-quantum/8853 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-pylint/8550 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":19},{"id":"b1f3e4f1a703ac5bf7222c96365adb0d6be3d0c9","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-20 03:04:09.000000000","message":"Patch Set 19:\n\nrecheck bug 1130451","accounts_in_message":[],"_revision_number":19},{"id":"2062625fc726b85c6b83fda345bdffc5313877cb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 04:04:42.000000000","message":"Patch Set 19:\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/19/check/gate-nova-pep8/21033 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-docs/15611 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-python26/17056 : FAILURE\n- http://logs.openstack.org/21481/19/check/gate-nova-python27/17111 : FAILURE\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-full/4374 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-postgres-full/4093 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-quantum/8880 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-pylint/8560 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":19},{"id":"f59760eefcbdfc89c496e06485ad644bbe01d527","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-20 04:50:32.000000000","message":"Patch Set 19:\n\nrecheck bug 1130451","accounts_in_message":[],"_revision_number":19},{"id":"6024a9707f7811031ba2214df42b18217b528088","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 05:47:15.000000000","message":"Patch Set 19: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/19/check/gate-nova-pep8/21050 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-docs/15626 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-python26/17071 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-python27/17126 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-full/4403 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-postgres-full/4122 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-tempest-devstack-vm-quantum/8912 : SUCCESS\n- http://logs.openstack.org/21481/19/check/gate-nova-pylint/8572 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":19},{"id":"84ee82dd118e9b40a425056319e7e0e3141b8c0b","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-02-20 14:43:54.000000000","message":"Patch Set 19: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":19},{"id":"e121518258e5ed06caefb79d3ff8b9c6a55cb62c","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-02-20 16:17:26.000000000","message":"Patch Set 19: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":19},{"id":"5efcbe1a0cc08031ac8f46b8fb316cb31ab6f1f9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 16:17:31.000000000","message":"Patch Set 19: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":19},{"id":"f23a98b009f62c902e40e2fc9f5497e7424cd2bf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 16:57:42.000000000","message":"Patch Set 19: Fails\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\nThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.","accounts_in_message":[],"_revision_number":19},{"id":"a67714a3afc94d9346c95378dabc3b0b39aedbde","author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"date":"2013-02-20 18:18:52.000000000","message":"Uploaded patch set 20.","accounts_in_message":[],"_revision_number":20},{"id":"040c62e2a09a483f849b899235096d684abcb8ed","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-02-20 18:27:51.000000000","message":"Patch Set 20: Looks good to me (core reviewer); Approved\n\nrekick rebased version","accounts_in_message":[],"_revision_number":20},{"id":"3b3de1f3e734b0f994817538c332b6021f233417","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 18:27:56.000000000","message":"Patch Set 20:\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":20},{"id":"336e32ef3f850b6f24b8b9cc5d00cbd17a571599","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 19:04:44.000000000","message":"Patch Set 20: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/21481/20/check/gate-nova-pep8/21129 : SUCCESS\n- http://logs.openstack.org/21481/20/check/gate-nova-docs/15700 : SUCCESS\n- http://logs.openstack.org/21481/20/check/gate-nova-python26/17145 : SUCCESS\n- http://logs.openstack.org/21481/20/check/gate-nova-python27/17200 : FAILURE\n- http://logs.openstack.org/21481/20/check/gate-tempest-devstack-vm-full/4559 : SUCCESS\n- http://logs.openstack.org/21481/20/check/gate-tempest-devstack-vm-postgres-full/4278 : SUCCESS\n- http://logs.openstack.org/21481/20/check/gate-tempest-devstack-vm-quantum/9088 : SUCCESS\n- http://logs.openstack.org/21481/20/check/gate-nova-pylint/8620 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":20},{"id":"03f7d69e3b0812df07bd9cee2c99b7d24265baed","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-20 19:10:16.000000000","message":"Patch Set 20: Works for me\n\nSmokeStack Results (patch set 20):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/73440\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/73441\n","accounts_in_message":[],"_revision_number":20},{"id":"34df8ceacae2c39145463c7686e39acdee360e7e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 20:03:29.000000000","message":"Patch Set 20: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/21481/20/gate/gate-nova-docs/15717 : SUCCESS\n- http://logs.openstack.org/21481/20/gate/gate-nova-pep8/21150 : SUCCESS\n- http://logs.openstack.org/21481/20/gate/gate-nova-python26/17162 : SUCCESS\n- http://logs.openstack.org/21481/20/gate/gate-nova-python27/17217 : SUCCESS\n- http://logs.openstack.org/21481/20/gate/gate-tempest-devstack-vm-full/4588 : SUCCESS\n- http://logs.openstack.org/21481/20/gate/gate-tempest-devstack-vm-postgres-full/4307 : SUCCESS\n- http://logs.openstack.org/21481/20/gate/gate-tempest-devstack-vm-quantum/9117 : SUCCESS\n","accounts_in_message":[],"_revision_number":20},{"id":"2b00622303e5147a0f570c0b478e0b0a57ab73e0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-20 20:03:32.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":20}],"current_revision_number":20,"current_revision":"c20110d15be37948ddd9ef5f38001328aabf5b1d","revisions":{"f8499dacb2114dedd73b19bfa1f320101b81537f":{"kind":"REWORK","_number":1,"created":"2013-02-08 00:52:31.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/1"}}},"commit":{"parents":[{"commit":"aa333c7a7cba7f26eb95735ff2eeeefe9f213608","subject":"Merge \"Default the last parameter (state) in task_log_get to None\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/aa333c7a7cba7f26eb95735ff2eeeefe9f213608"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-08 00:51:26.000000000","tz":-480},"subject":"Add status and instance_uuid to bm_deployments","message":"Add status and instance_uuid to bm_deployments\n\nAdd \u0027status\u0027 and \u0027instance_uuid\u0027 columns to bm_deployments.\n\nThis patch changes the behaviour of bm_deployments: they are no\nlonger deleted immediately after completion. Instead, a deployment is\nmarked as ACTIVE. Later, when an instance is deleted, the associated\ndeployment is also deleted.\n\nIn this way, an active baremetal instance can be verified by comparing\nPXE/TFTP configuration with an active entry in bm_deployments. This will be\nuseful in identifying orphaned instances and zombie nodes\n(to be done in a future patch).\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f8499dacb2114dedd73b19bfa1f320101b81537f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f8499dacb2114dedd73b19bfa1f320101b81537f"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"aa333c7a7cba7f26eb95735ff2eeeefe9f213608","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"4e8fe76ba65b80d7a9fe3127d2b690381760ce79":{"kind":"REWORK","_number":2,"created":"2013-02-08 06:38:42.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/2"}}},"commit":{"parents":[{"commit":"64106c282f961d65ca469524016635c96596b77f","subject":"Merge \"Optimize rpc handling for allocate and deallocate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/64106c282f961d65ca469524016635c96596b77f"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-08 06:37:32.000000000","tz":-480},"subject":"Add status and instance_uuid to bm_deployments","message":"Add status and instance_uuid to bm_deployments\n\nAdd \u0027status\u0027 and \u0027instance_uuid\u0027 columns to bm_deployments.\n\nThis patch changes the behaviour of bm_deployments: they are no\nlonger deleted immediately after completion. Instead, a deployment is\nmarked as ACTIVE. Later, when an instance is deleted, the associated\ndeployment is also deleted.\n\nIn this way, an active baremetal instance can be verified by comparing\nPXE/TFTP configuration with an active entry in bm_deployments. This will be\nuseful in identifying orphaned instances and zombie nodes\n(to be done in a future patch).\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4e8fe76ba65b80d7a9fe3127d2b690381760ce79"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4e8fe76ba65b80d7a9fe3127d2b690381760ce79"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"64106c282f961d65ca469524016635c96596b77f","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"fd0b5ebe395d7cf921582223ae67c9e038f1ed50":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2013-02-08 06:40:27.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/3"}}},"commit":{"parents":[{"commit":"64106c282f961d65ca469524016635c96596b77f","subject":"Merge \"Optimize rpc handling for allocate and deallocate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/64106c282f961d65ca469524016635c96596b77f"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-08 06:38:17.000000000","tz":-480},"subject":"Add status and instance_uuid to bm_deployments","message":"Add status and instance_uuid to bm_deployments\n\nAdd \u0027status\u0027 and \u0027instance_uuid\u0027 columns to bm_deployments.\n\nThis patch changes the behaviour of bm_deployments: they are no\nlonger deleted immediately after completion. Instead, a deployment is\nmarked as ACTIVE. Later, when an instance is deleted, the associated\ndeployment is also deleted.\n\nIn this way, an active baremetal instance can be verified by comparing\nPXE/TFTP configuration with an active entry in bm_deployments. This will be\nuseful in identifying orphaned instances and zombie nodes\n(to be done in a future patch).\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/fd0b5ebe395d7cf921582223ae67c9e038f1ed50"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/fd0b5ebe395d7cf921582223ae67c9e038f1ed50"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"64106c282f961d65ca469524016635c96596b77f","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"96f2ce3fc24e6ccb69d034f9234a5690a2dbb2a8":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2013-02-09 09:41:21.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/4"}}},"commit":{"parents":[{"commit":"9baabd3fec8e2a4849be25d1f48903aa61176119","subject":"Merge \"Use joined version of db.api calls\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9baabd3fec8e2a4849be25d1f48903aa61176119"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-09 05:42:12.000000000","tz":-480},"subject":"Add status and instance_uuid to bm_deployments","message":"Add status and instance_uuid to bm_deployments\n\nAdd \u0027status\u0027 and \u0027instance_uuid\u0027 columns to bm_deployments.\n\nThis patch changes the behaviour of bm_deployments: they are no\nlonger deleted immediately after completion. Instead, a deployment is\nmarked as ACTIVE. Later, when an instance is deleted, the associated\ndeployment is also deleted.\n\nIn this way, an active baremetal instance can be verified by comparing\nPXE/TFTP configuration with an active entry in bm_deployments. This will be\nuseful in identifying orphaned instances and zombie nodes\n(to be done in a future patch).\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/96f2ce3fc24e6ccb69d034f9234a5690a2dbb2a8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/96f2ce3fc24e6ccb69d034f9234a5690a2dbb2a8"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"9baabd3fec8e2a4849be25d1f48903aa61176119","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"9c1b963e0502ed797e9362c0a72211e27af4a44a":{"kind":"REWORK","_number":5,"created":"2013-02-10 17:15:13.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/5"}}},"commit":{"parents":[{"commit":"9baabd3fec8e2a4849be25d1f48903aa61176119","subject":"Merge \"Use joined version of db.api calls\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9baabd3fec8e2a4849be25d1f48903aa61176119"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-10 17:01:07.000000000","tz":-480},"subject":"Add status and instance_uuid to bm_deployments","message":"Add status and instance_uuid to bm_deployments\n\nAdd \u0027status\u0027 and \u0027instance_uuid\u0027 columns and an index on \u0027bm_node_id\u0027\nto bm_deployments.\n\nThis patch changes the behaviour of bm_deployments: they are no\nlonger deleted immediately after completion. Instead, a deployment is\nmarked as ACTIVE. Later, when an instance is deleted, the associated\ndeployment is also deleted.\n\nIn this way, an active baremetal instance can be verified by comparing\nPXE/TFTP configuration with an active entry in bm_deployments. This will be\nuseful in identifying orphaned instances and zombie nodes\n(to be done in a future patch).\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9c1b963e0502ed797e9362c0a72211e27af4a44a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9c1b963e0502ed797e9362c0a72211e27af4a44a"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"9baabd3fec8e2a4849be25d1f48903aa61176119","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"cf1a7821ac3318ef1569cde11a089cd83627154a":{"kind":"REWORK","_number":6,"created":"2013-02-13 02:42:34.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/6"}}},"commit":{"parents":[{"commit":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","subject":"Don\u0027t modify injected_files inside PXE driver","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263591b093921646cdd1bf7a1bcf6f7fa8c81dc3"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-13 02:38:48.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/cf1a7821ac3318ef1569cde11a089cd83627154a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/cf1a7821ac3318ef1569cde11a089cd83627154a"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","is_merged_in_target_branch":false,"change_id":"I508cfc0baec15a091a1717eb2aebb9dd39278e5b","change_number":21846,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"5f3457c350a76a8f20b6014df647399fd6cf2f7d":{"kind":"REWORK","_number":7,"created":"2013-02-13 05:31:33.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/7"}}},"commit":{"parents":[{"commit":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","subject":"Don\u0027t modify injected_files inside PXE driver","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263591b093921646cdd1bf7a1bcf6f7fa8c81dc3"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-13 05:24:21.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5f3457c350a76a8f20b6014df647399fd6cf2f7d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5f3457c350a76a8f20b6014df647399fd6cf2f7d"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","is_merged_in_target_branch":false,"change_id":"I508cfc0baec15a091a1717eb2aebb9dd39278e5b","change_number":21846,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"195548c4d30c2e9682a7f05e066c524d09c402d7":{"kind":"REWORK","_number":8,"created":"2013-02-13 05:57:39.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/8"}}},"commit":{"parents":[{"commit":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","subject":"Don\u0027t modify injected_files inside PXE driver","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263591b093921646cdd1bf7a1bcf6f7fa8c81dc3"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-13 05:53:15.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/195548c4d30c2e9682a7f05e066c524d09c402d7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/195548c4d30c2e9682a7f05e066c524d09c402d7"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","is_merged_in_target_branch":false,"change_id":"I508cfc0baec15a091a1717eb2aebb9dd39278e5b","change_number":21846,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"f44b7c5e94c5c0b78c9c0826d2937f58dac8f861":{"kind":"REWORK","_number":9,"created":"2013-02-13 19:41:31.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/9"}}},"commit":{"parents":[{"commit":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","subject":"Don\u0027t modify injected_files inside PXE driver","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263591b093921646cdd1bf7a1bcf6f7fa8c81dc3"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-13 19:38:23.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test_baremetal_migration.py script, based on\ntest_migrations.py.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f44b7c5e94c5c0b78c9c0826d2937f58dac8f861"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f44b7c5e94c5c0b78c9c0826d2937f58dac8f861"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","is_merged_in_target_branch":false,"change_id":"I508cfc0baec15a091a1717eb2aebb9dd39278e5b","change_number":21846,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"e0139444d1c8d3d9fd4ae0348e1b4c61f2208ed9":{"kind":"REWORK","_number":10,"created":"2013-02-13 21:09:28.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/10"}}},"commit":{"parents":[{"commit":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","subject":"Don\u0027t modify injected_files inside PXE driver","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263591b093921646cdd1bf7a1bcf6f7fa8c81dc3"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-13 21:07:32.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test_baremetal_migration.py script, based on\ntest_migrations.py.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e0139444d1c8d3d9fd4ae0348e1b4c61f2208ed9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e0139444d1c8d3d9fd4ae0348e1b4c61f2208ed9"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","is_merged_in_target_branch":false,"change_id":"I508cfc0baec15a091a1717eb2aebb9dd39278e5b","change_number":21846,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"c96faf7162d9206614e7e9f24ba3c8b1d18d85cf":{"kind":"REWORK","_number":11,"created":"2013-02-14 03:15:35.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/11"}}},"commit":{"parents":[{"commit":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","subject":"Don\u0027t modify injected_files inside PXE driver","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263591b093921646cdd1bf7a1bcf6f7fa8c81dc3"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-14 03:12:36.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test_baremetal_migration.py script, based on\ntest_migrations.py.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c96faf7162d9206614e7e9f24ba3c8b1d18d85cf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c96faf7162d9206614e7e9f24ba3c8b1d18d85cf"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"263591b093921646cdd1bf7a1bcf6f7fa8c81dc3","is_merged_in_target_branch":false,"change_id":"I508cfc0baec15a091a1717eb2aebb9dd39278e5b","change_number":21846,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"7657167b75f736df4cfcb7b2b84cc33f14b1d427":{"kind":"REWORK","_number":12,"created":"2013-02-14 14:40:44.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/12"}}},"commit":{"parents":[{"commit":"e332c8a3133b1c550c02f8793c19fc2ee062a127","subject":"Don\u0027t modify injected_files inside PXE driver","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e332c8a3133b1c550c02f8793c19fc2ee062a127"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-14 14:36:57.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test_baremetal_migration.py script, based on\ntest_migrations.py.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7657167b75f736df4cfcb7b2b84cc33f14b1d427"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7657167b75f736df4cfcb7b2b84cc33f14b1d427"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"e332c8a3133b1c550c02f8793c19fc2ee062a127","is_merged_in_target_branch":true,"change_id":"I508cfc0baec15a091a1717eb2aebb9dd39278e5b","change_number":21846,"patch_set_number":2,"change_status":"MERGED"}],"branch":"refs/heads/master"},"f710bd9a4528a0b1e01e8b476d03ed900e82034d":{"kind":"REWORK","_number":13,"created":"2013-02-15 18:16:53.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/13"}}},"commit":{"parents":[{"commit":"6cab9c3d1c5b31284c3da5f531ab879bd27e3496","subject":"Merge \"Don\u0027t modify injected_files inside PXE driver\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6cab9c3d1c5b31284c3da5f531ab879bd27e3496"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-15 18:12:32.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test_baremetal_migration.py script, based on\ntest_migrations.py.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f710bd9a4528a0b1e01e8b476d03ed900e82034d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f710bd9a4528a0b1e01e8b476d03ed900e82034d"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"6cab9c3d1c5b31284c3da5f531ab879bd27e3496","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"9621daff783a8aa9b6491d046e35ba75cb5d0873":{"kind":"TRIVIAL_REBASE","_number":14,"created":"2013-02-18 22:27:30.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/14","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/14","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/14 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/14 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/14 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/14"}}},"commit":{"parents":[{"commit":"b23c557cc8d03028caba95777f98adfe9b1031d9","subject":"Merge \"Enable multipath for libvirt iSCSI Volume Driver\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b23c557cc8d03028caba95777f98adfe9b1031d9"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-18 22:27:04.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test_baremetal_migration.py script, based on\ntest_migrations.py.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9621daff783a8aa9b6491d046e35ba75cb5d0873"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9621daff783a8aa9b6491d046e35ba75cb5d0873"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"b23c557cc8d03028caba95777f98adfe9b1031d9","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"d5aa04c7a752b53b8cc021d6c172f4b9b4932490":{"kind":"REWORK","_number":15,"created":"2013-02-19 19:59:05.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/15","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/15","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/15 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/15 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/15 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/15"}}},"commit":{"parents":[{"commit":"b23c557cc8d03028caba95777f98adfe9b1031d9","subject":"Merge \"Enable multipath for libvirt iSCSI Volume Driver\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b23c557cc8d03028caba95777f98adfe9b1031d9"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-06 07:35:23.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 19:57:02.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test class, TestBaremetalMigrations, and refactors the\ntest_migrations.py file to allow for multiple test classes.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d5aa04c7a752b53b8cc021d6c172f4b9b4932490"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d5aa04c7a752b53b8cc021d6c172f4b9b4932490"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"b23c557cc8d03028caba95777f98adfe9b1031d9","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"1a0b6875744cca8921335882e1de45c469a2b3c3":{"kind":"REWORK","_number":16,"created":"2013-02-19 20:42:34.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/16","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/16","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/16 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/16 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/16 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/16"}}},"commit":{"parents":[{"commit":"f237685e8a2f63c697c60adfe8206abbe5beca45","subject":"Merge \"Stub additional FloatingIP methods in FlatManager\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f237685e8a2f63c697c60adfe8206abbe5beca45"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 20:39:55.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 20:39:55.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test class, TestBaremetalMigrations, and refactors the\ntest_migrations.py file to allow for multiple test classes.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1a0b6875744cca8921335882e1de45c469a2b3c3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1a0b6875744cca8921335882e1de45c469a2b3c3"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"f237685e8a2f63c697c60adfe8206abbe5beca45","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"47920c9bd3a676587817cabf2cbfca0de0ae6893":{"kind":"REWORK","_number":17,"created":"2013-02-19 20:59:17.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/17","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/17","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/17 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/17 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/17 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/17"}}},"commit":{"parents":[{"commit":"f237685e8a2f63c697c60adfe8206abbe5beca45","subject":"Merge \"Stub additional FloatingIP methods in FlatManager\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f237685e8a2f63c697c60adfe8206abbe5beca45"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 20:39:55.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 20:58:28.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test class, TestBaremetalMigrations, and refactors the\ntest_migrations.py file to allow for multiple test classes.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/47920c9bd3a676587817cabf2cbfca0de0ae6893"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/47920c9bd3a676587817cabf2cbfca0de0ae6893"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"f237685e8a2f63c697c60adfe8206abbe5beca45","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"a0e31b5f16dd52a1cac5b7c2ad2e6b3440bb73eb":{"kind":"REWORK","_number":18,"created":"2013-02-19 22:42:23.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/18","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/18","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/18 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/18 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/18 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/18"}}},"commit":{"parents":[{"commit":"f237685e8a2f63c697c60adfe8206abbe5beca45","subject":"Merge \"Stub additional FloatingIP methods in FlatManager\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f237685e8a2f63c697c60adfe8206abbe5beca45"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 20:39:55.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 22:40:28.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test class, TestBaremetalMigrations, and refactors the\ntest_migrations.py file to allow for multiple test classes.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a0e31b5f16dd52a1cac5b7c2ad2e6b3440bb73eb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a0e31b5f16dd52a1cac5b7c2ad2e6b3440bb73eb"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"f237685e8a2f63c697c60adfe8206abbe5beca45","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"3ad373b02c1a6cc812451b18664bc9bd4ca6f3d7":{"kind":"REWORK","_number":19,"created":"2013-02-19 23:10:57.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/19","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/19","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/19 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/19 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/19 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/19"}}},"commit":{"parents":[{"commit":"f237685e8a2f63c697c60adfe8206abbe5beca45","subject":"Merge \"Stub additional FloatingIP methods in FlatManager\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f237685e8a2f63c697c60adfe8206abbe5beca45"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 20:39:55.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 23:09:39.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test class, TestBaremetalMigrations, and refactors the\ntest_migrations.py file to allow for multiple test classes.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3ad373b02c1a6cc812451b18664bc9bd4ca6f3d7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3ad373b02c1a6cc812451b18664bc9bd4ca6f3d7"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"f237685e8a2f63c697c60adfe8206abbe5beca45","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"c20110d15be37948ddd9ef5f38001328aabf5b1d":{"kind":"REWORK","_number":20,"created":"2013-02-20 18:18:52.000000000","uploader":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"ref":"refs/changes/81/21481/20","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/21481/20","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/20 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/20 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/21481/20 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/21481/20"}}},"commit":{"parents":[{"commit":"666cb4eaf70c9f2f8dd05ba0df91f0e7dd6810cb","subject":"Merge \"Add API Sample tests for Hypervisors extension.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/666cb4eaf70c9f2f8dd05ba0df91f0e7dd6810cb"}]}],"author":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-19 20:39:55.000000000","tz":-480},"committer":{"name":"Devananda van der Veen","email":"devananda.vdv@gmail.com","date":"2013-02-20 17:55:02.000000000","tz":-480},"subject":"Add better status to baremetal deployments.","message":"Add better status to baremetal deployments.\n\nThis patch introduces a few new baremetal states, which are used to\ntrack the deploy process. Now, nova-baremetal-deploy-helper updates the\nbm_nodes record directly when it begins and finishes deploying an image\nto that node.\n\nThe next patch will add a LoopingCall inside driver.spawn() to wait for\nthe deploy to complete.\n\nAlso, since there can not be \u003e1 active deployment per node, there\nis no need to have a separate table for storing them. This patch drops\nthe table bm_deployments and adds the important information it contained\nto bm_nodes. Since the previous behavior was to mark a deployment as\ndeleted once it completed, there is no need to copy any data from\nbm_deployments prior to dropping the table -- assuming that no active\ndeployments are in process when the migration is run.\n\nSince this is the first migration for the baremetal database, it also\nadds a new test class, TestBaremetalMigrations, and refactors the\ntest_migrations.py file to allow for multiple test classes.\n\npartially implements fix for bug 1096723\n\nChange-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c20110d15be37948ddd9ef5f38001328aabf5b1d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c20110d15be37948ddd9ef5f38001328aabf5b1d"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"666cb4eaf70c9f2f8dd05ba0df91f0e7dd6810cb","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}