)]}'
{"id":"openstack%2Fnova~28403","triplet_id":"openstack%2Fnova~master~Ied2bb1eeb0cbdc6903a4028a60c89ca9e23656b4","project":"openstack/nova","branch":"master","topic":"update-processutils-20130507","hashtags":[],"change_id":"Ied2bb1eeb0cbdc6903a4028a60c89ca9e23656b4","subject":"Convert to using newly imported processutils.","status":"MERGED","created":"2013-05-07 11:08:41.000000000","updated":"2013-05-13 19:16:52.000000000","submitted":"2013-05-13 19:16:49.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a1e683b4a7a78c74cad4dc5d3029f175175d1261","_number":28403,"virtual_id_number":28403,"owner":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"recommended":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":1,"date":"2013-05-07 20:27:46.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},{"value":2,"date":"2013-05-13 19:16:49.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"all":[{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":2,"date":"2013-05-11 01:02:16.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":2,"date":"2013-05-08 08:00:05.000000000","_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"all":[{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":1,"date":"2013-05-11 01:02:16.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"date":"2013-05-08 08:00:05.000000000","_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-05-07 11:32:00.000000000","updated_by":{"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},"reviewer":{"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},"state":"REVIEWER"},{"updated":"2013-05-07 13:41:12.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-05-07 20:27:46.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-05-08 08:00:05.000000000","updated_by":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"reviewer":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"state":"REVIEWER"},{"updated":"2013-05-11 01:02:16.000000000","updated_by":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"reviewer":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"state":"REVIEWER"},{"updated":"2013-05-13 19:16:49.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5a5ef77aab7a5bfaa6a9739cce906477cbd8d190","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-05-07 11:28:15.000000000","message":"Patch Set 1: Works for me\n\nSmokeStack Results (patch set 1):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/85727\n","accounts_in_message":[],"_revision_number":1},{"id":"eec73037363424bdbf6f0394f3d2f305e5ac953b","author":{"_account_id":4419,"name":"Matthew Sherborne","email":"msherborne+openstack@gmail.com","username":"msherborne+openstack"},"date":"2013-05-07 11:32:00.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nLooking good so far..","accounts_in_message":[],"_revision_number":1},{"id":"c93187b748f25fe49378191013b457e328827be2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-07 11:54:05.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/28403/1/check/gate-nova-pep8/25447 : SUCCESS in 00:03:10\n- http://docs-draft.openstack.org/28403/1/check/gate-nova-docs/19665/doc/build/html/ : SUCCESS in 00:06:09\n- http://logs.openstack.org/28403/1/check/gate-nova-python26/21073 : SUCCESS in 00:18:41\n- http://logs.openstack.org/28403/1/check/gate-nova-python26-centos6/279 : SUCCESS in 00:12:26 (non-voting)\n- http://logs.openstack.org/28403/1/check/gate-nova-python27/21161 : SUCCESS in 00:08:28\n- http://logs.openstack.org/28403/1/check/gate-tempest-devstack-vm-full/14816 : SUCCESS in 00:44:32\n- http://logs.openstack.org/28403/1/check/gate-tempest-devstack-vm-postgres-full/14531 : FAILURE in 00:43:59 (non-voting)\n- http://logs.openstack.org/28403/1/check/gate-tempest-devstack-vm-quantum/21019 : SUCCESS in 00:21:37\n- http://logs.openstack.org/28403/1/check/gate-grenade-devstack-vm/2595 : SUCCESS in 00:14:21 (non-voting)\n- http://logs.openstack.org/28403/1/check/gate-nova-pylint/11165 : SUCCESS in 00:08:40 (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"aec3ed5035d175e04f37e145ed49e76d56d95403","author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"date":"2013-05-07 13:41:12.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"a36bfffe043abd57e030dc6432871f70161d217d","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2013-05-07 19:55:07.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"18dcac57e7ed7f3f8c5118f8eb2dfc546d402d09","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2013-05-07 20:00:36.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"a3829da93a67bf13e6a3099310bd620a65de4faa","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-05-07 20:27:46.000000000","message":"Patch Set 2: Works for me\n\nSmokeStack Results (patch set 2):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/85767\n","accounts_in_message":[],"_revision_number":2},{"id":"fbf3a7b8fe420abb6126fd0f6017f7c1ca9ba015","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-07 20:46:24.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/28403/2/check/gate-nova-pep8/25473 : SUCCESS in 00:03:04\n- http://docs-draft.openstack.org/28403/2/check/gate-nova-docs/19691/doc/build/html/ : SUCCESS in 00:06:23\n- http://logs.openstack.org/28403/2/check/gate-nova-python26/21099 : SUCCESS in 00:19:23\n- http://logs.openstack.org/28403/2/check/gate-nova-python26-centos6/303 : SUCCESS in 00:13:19 (non-voting)\n- http://logs.openstack.org/28403/2/check/gate-nova-python27/21187 : SUCCESS in 00:08:33\n- http://logs.openstack.org/28403/2/check/gate-tempest-devstack-vm-full/14896 : SUCCESS in 00:44:08\n- http://logs.openstack.org/28403/2/check/gate-tempest-devstack-vm-postgres-full/14611 : FAILURE in 00:44:56 (non-voting)\n- http://logs.openstack.org/28403/2/check/gate-tempest-devstack-vm-quantum/21116 : SUCCESS in 00:20:46\n- http://logs.openstack.org/28403/2/check/gate-grenade-devstack-vm/2663 : SUCCESS in 00:14:24 (non-voting)\n- http://logs.openstack.org/28403/2/check/gate-nova-pylint/11189 : SUCCESS in 00:08:37 (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"0717c0f1274098cb4e35619c552ce7455c42f5e6","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-05-08 08:00:05.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nlgtm... when dependent patch lands :)","accounts_in_message":[],"_revision_number":2},{"id":"dd65722909d9ba257a431dc8ae4c300d593306ae","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2013-05-11 01:02:16.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"4dc36b832f7cd20c4e5e3038fe4d2b2e4e5e5c96","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-13 18:23:52.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"8582ad929e718e39f7320ca2e7264c58c8605a51","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-13 19:16:49.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://docs-draft.openstack.org/28403/2/gate/gate-nova-docs/19992/doc/build/html/ : SUCCESS in 00:02:32\n- http://logs.openstack.org/28403/2/gate/gate-nova-pep8/25774 : SUCCESS in 00:03:04\n- http://logs.openstack.org/28403/2/gate/gate-nova-python26/21400 : SUCCESS in 00:12:39\n- http://logs.openstack.org/28403/2/gate/gate-nova-python27/21488 : SUCCESS in 00:10:12\n- http://logs.openstack.org/28403/2/gate/gate-tempest-devstack-vm-full/15733 : SUCCESS in 00:45:21\n- http://logs.openstack.org/28403/2/gate/gate-tempest-devstack-vm-postgres-full/15449 : FAILURE in 00:51:59 (non-voting)\n- http://logs.openstack.org/28403/2/gate/gate-tempest-devstack-vm-quantum/22139 : SUCCESS in 00:21:42\n- http://logs.openstack.org/28403/2/gate/gate-grenade-devstack-vm/3397 : FAILURE in 00:11:41 (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"3c0a75945b87e800461b42dc6dcad41d837529f2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-13 19:16:52.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"6be85774aec674eef7bfbf66f8885a4531c7c605","revisions":{"dadd5362cb254aeca59066fc7af2ca526166f4db":{"kind":"REWORK","_number":1,"created":"2013-05-07 11:08:41.000000000","uploader":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"ref":"refs/changes/03/28403/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/03/28403/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/03/28403/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/03/28403/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/03/28403/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/03/28403/1"}}},"commit":{"parents":[{"commit":"1e26360a5e9a0407cb497eae598976ea6d3b76f3","subject":"Import new additions to oslo\u0027s processutils.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1e26360a5e9a0407cb497eae598976ea6d3b76f3"}]}],"author":{"name":"Michael Still","email":"mikal@stillhq.com","date":"2013-05-07 10:51:24.000000000","tz":600},"committer":{"name":"Michael Still","email":"mikal@stillhq.com","date":"2013-05-07 11:06:11.000000000","tz":600},"subject":"Convert to using newly imported processutils.","message":"Convert to using newly imported processutils.\n\nThis change moves nova over to using the newly imported trycmd() and\nssh_execute() implementations from oslo\u0027s processutils.\n\nChange-Id: Ied2bb1eeb0cbdc6903a4028a60c89ca9e23656b4\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dadd5362cb254aeca59066fc7af2ca526166f4db"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dadd5362cb254aeca59066fc7af2ca526166f4db"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"1e26360a5e9a0407cb497eae598976ea6d3b76f3","is_merged_in_target_branch":false,"change_id":"I69926af33ff824f712ae2b8e4023233afe1dff2f","change_number":28402,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"6be85774aec674eef7bfbf66f8885a4531c7c605":{"kind":"REWORK","_number":2,"created":"2013-05-07 20:00:36.000000000","uploader":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"ref":"refs/changes/03/28403/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/03/28403/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/03/28403/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/03/28403/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/03/28403/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/03/28403/2"}}},"commit":{"parents":[{"commit":"19718560a7c3f6b1335dde0b62b544ee4bc83f58","subject":"Import new additions to oslo\u0027s processutils.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/19718560a7c3f6b1335dde0b62b544ee4bc83f58"}]}],"author":{"name":"Michael Still","email":"mikal@stillhq.com","date":"2013-05-07 10:51:24.000000000","tz":600},"committer":{"name":"Michael Still","email":"mikal@stillhq.com","date":"2013-05-07 19:55:27.000000000","tz":600},"subject":"Convert to using newly imported processutils.","message":"Convert to using newly imported processutils.\n\nThis change moves nova over to using the newly imported trycmd() and\nssh_execute() implementations from oslo\u0027s processutils.\n\nChange-Id: Ied2bb1eeb0cbdc6903a4028a60c89ca9e23656b4\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6be85774aec674eef7bfbf66f8885a4531c7c605"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6be85774aec674eef7bfbf66f8885a4531c7c605"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"19718560a7c3f6b1335dde0b62b544ee4bc83f58","is_merged_in_target_branch":true,"change_id":"I69926af33ff824f712ae2b8e4023233afe1dff2f","change_number":28402,"patch_set_number":2,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}