)]}'
{"id":"openstack%2Fdevstack~53240","triplet_id":"openstack%2Fdevstack~master~I279732311cfe1cb1f73c1a2b1bec2005f9a43bf0","project":"openstack/devstack","branch":"master","topic":"sequence-change","hashtags":[],"change_id":"I279732311cfe1cb1f73c1a2b1bec2005f9a43bf0","subject":"change order of cleanup_* and install_*","status":"ABANDONED","created":"2013-10-22 22:21:23.000000000","updated":"2013-12-14 06:03:22.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"1aa6cb471e89f6c1fd9944c935354c2154a181f6","_number":53240,"virtual_id_number":53240,"owner":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":1,"date":"2013-10-30 12:43:32.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":{"disliked":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"date":"2013-10-23 00:52:40.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":-1,"date":"2013-12-06 11:23:51.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2013-11-17 20:54:45.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"date":"2013-12-06 11:23:51.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-30 12:43:32.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-11-17 20:54:45.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-12-06 11:23:51.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"}],"messages":[{"id":"015da3c4d29673b0bba7cc78ea8a6c6bfe39e3b4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-22 22:51:13.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/40/53240/1/check/gate-devstack-bash8/94cb37b : SUCCESS in 4s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/40/53240/1/check/check-tempest-devstack-vm-full/0a8edc8 : SUCCESS in 29m 39s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/40/53240/1/check/check-tempest-devstack-vm-postgres-full/0222723 : SUCCESS in 29m 42s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/40/53240/1/check/check-tempest-devstack-vm-neutron/c71db97 : FAILURE in 18m 00s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/40/53240/1/check/gate-tempest-devstack-vm-large-ops/d153248 : SUCCESS in 14m 18s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/40/53240/1/check/gate-tempest-devstack-vm-neutron-large-ops/26e27b1 : SUCCESS in 16m 14s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/40/53240/1/check/check-grenade-devstack-vm/496a25f : SUCCESS in 24m 13s\n- check-swift-devstack-vm-functional http://logs.openstack.org/40/53240/1/check/check-swift-devstack-vm-functional/6ce4515 : SUCCESS in 10m 56s\n","accounts_in_message":[],"_revision_number":1},{"id":"3391a9d19602bf2b4f8425c3cdbf64855cddf62c","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-23 00:52:40.000000000","message":"Patch Set 1:\n\nrecheck nobug","accounts_in_message":[],"_revision_number":1},{"id":"f65534b45c6f958203aaf3ec4d0d6cc4bc0dda3c","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-30 06:03:02.000000000","message":"Patch Set 1: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"9cf9fd4965f88f41c328b79390959a5d310936f6","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-30 06:03:02.000000000","message":"Patch Set 1:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"0298312fdc37c26c8a616eba2e59976282e81ff2","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-30 12:09:06.000000000","message":"Patch Set 1: Restored","accounts_in_message":[],"_revision_number":1},{"id":"1100d9bacd78d92ea44203f5f289e5381249a80f","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-30 12:09:24.000000000","message":"Patch Set 1:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":1},{"id":"b0c5f52340d3c4dfcfb83f13fa9a193ae4a9539f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-30 12:43:32.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/40/53240/1/check/gate-devstack-bash8/d86a955 : SUCCESS in 5s\n- check-tempest-devstack-vm-full http://logs.openstack.org/40/53240/1/check/check-tempest-devstack-vm-full/10e53d3 : SUCCESS in 33m 23s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/40/53240/1/check/check-tempest-devstack-vm-postgres-full/a4f8021 : SUCCESS in 34m 14s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/40/53240/1/check/check-tempest-devstack-vm-neutron/04f6f41 : SUCCESS in 18m 52s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/40/53240/1/check/gate-tempest-devstack-vm-large-ops/47836dd : SUCCESS in 18m 08s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/40/53240/1/check/gate-tempest-devstack-vm-neutron-large-ops/a31dd11 : SUCCESS in 17m 34s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/40/53240/1/check/check-grenade-devstack-vm/7279140 : SUCCESS in 24m 02s\n- check-swift-devstack-vm-functional http://logs.openstack.org/40/53240/1/check/check-swift-devstack-vm-functional/4b52072 : SUCCESS in 11m 50s\n","accounts_in_message":[],"_revision_number":1},{"id":"d6ca8661463e5a9a573811582c85f3f8350a3dc8","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-12-06 11:23:51.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nI don\u0027t feel that the commit message sufficiently explains the need for this change. What breaks if we don\u0027t do it?","accounts_in_message":[],"_revision_number":1},{"id":"cb770460a84fa9f9631bf3b3b8b09ef03df0ad80","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-12-14 06:03:22.000000000","message":"Patch Set 1: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"03059c0f6c0facb906721832968f225cda0ebf80","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-12-14 06:03:22.000000000","message":"Patch Set 1:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"c65646e57589736483256849e80f3a5ec5f175b4","revisions":{"c65646e57589736483256849e80f3a5ec5f175b4":{"kind":"REWORK","_number":1,"created":"2013-10-22 22:21:23.000000000","uploader":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"ref":"refs/changes/40/53240/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/40/53240/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/53240/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/53240/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/53240/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/40/53240/1"}}},"commit":{"parents":[{"commit":"80c3ea52ed1caec16e5c6c6beecd434225afbd97","subject":"Merge \"exit cleanup in functions\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/80c3ea52ed1caec16e5c6c6beecd434225afbd97"}]}],"author":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-22 22:21:19.000000000","tz":-300},"committer":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-22 22:21:24.000000000","tz":-300},"subject":"change order of cleanup_* and install_*","message":"change order of cleanup_* and install_*\n\nBy logical, we shall first cleanup_*, install_*.\nThe two steps actually have no dependency,\nthus we shall be safe for the reorder\n\nChange-Id: I279732311cfe1cb1f73c1a2b1bec2005f9a43bf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c65646e57589736483256849e80f3a5ec5f175b4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c65646e57589736483256849e80f3a5ec5f175b4"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}