)]}'
{"id":"openstack%2Fnova~684","triplet_id":"openstack%2Fnova~master~Ibc1fec34c7aee40c1252da8a3d174683434269c1","project":"openstack/nova","branch":"master","topic":"api_ec2_admin_test","hashtags":[],"change_id":"Ibc1fec34c7aee40c1252da8a3d174683434269c1","subject":"api.ec2.admin unit tests","status":"MERGED","created":"2011-09-27 15:03:29.000000000","updated":"2011-10-13 05:58:40.000000000","submitted":"2011-10-13 05:58:39.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":6,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"886048a0878f6e871ab01c70e8c7337b9f594685","_number":684,"virtual_id_number":684,"owner":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2011-10-13 05:58:39.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},{"value":0,"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"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":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"all":[{"value":0,"date":"2011-10-13 05:18:57.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2011-10-13 05:18:21.000000000","_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},{"value":1,"date":"2011-09-30 15:28:30.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"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":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},{"value":0,"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"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":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},{"value":0,"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"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":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2011-09-28 09:59:03.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":"2011-09-30 15:28:30.000000000","updated_by":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"reviewer":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"state":"REVIEWER"},{"updated":"2011-10-13 05:18:21.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":"2011-10-13 05:58:39.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"6f1ad58472dbd781ab91f09b73a49b47fe09c023","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2011-09-28 06:42:39.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n(3 inline comments)\n\nLooks great to me, very comprehensive.\n\nThere\u0027s just a few things I noticed which I commented on inline. None of them are blockers though.","accounts_in_message":[],"_revision_number":1},{"id":"d76590ea4d7a67aa1b18beaa868e16802216130b","author":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"date":"2011-09-28 09:06:57.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"4851256e49d44520c3d3a68f8084cb59115ce5a5","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2011-09-28 09:59:03.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nLooks great","accounts_in_message":[],"_revision_number":2},{"id":"8eeead00b34ffae4f49d2a7ee45beb55367cf925","author":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"date":"2011-09-28 10:01:55.000000000","message":"Patch Set 1: (3 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"27a057f498f12a52f3567a43b562016b91654f4f","author":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"date":"2011-09-29 12:48:01.000000000","message":"Patch Set 2:\n\nMark brought to my attention in another review that Python 2.6 doesn\u0027t work with:\n\nwith self.assertRaises(...):\n    \u003cfunc to test\u003e\n\nWill patch and submit","accounts_in_message":[],"_revision_number":2},{"id":"3f7baff924e89cfb68e9f7b0a2269254df74c058","author":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"date":"2011-09-29 12:57:11.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"74f2bd4899ec8506fadcf43b51a7077c8ff7d29b","author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"date":"2011-09-30 15:28:30.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\nAwesome patch","accounts_in_message":[],"_revision_number":3},{"id":"5e3b262f039a4a9f48d07194ad61552061661d2f","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2011-10-13 05:18:21.000000000","message":"Patch Set 3: Looks good to me, approved\n\nnice!","accounts_in_message":[],"_revision_number":3},{"id":"5a3a6e6b04e4d16d27e0ab7a48e9328aff6f60c0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:18:57.000000000","message":"Patch Set 3:\n\nBuild Started https://jenkins.openstack.org/job/nova-merge/117/ (1/3)","accounts_in_message":[],"_revision_number":3},{"id":"61fa6b8e44284a6a91e985c26236332cea9d4afe","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:19:27.000000000","message":"Patch Set 3:\n\nBuild Started https://jenkins.openstack.org/job/nova-pep8/1489/ (2/3)","accounts_in_message":[],"_revision_number":3},{"id":"a52b238361582f115e36d4b433978eacd1a93a58","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:29:31.000000000","message":"Patch Set 3:\n\nBuild Started https://jenkins.openstack.org/job/nova/121639/ (3/3)","accounts_in_message":[],"_revision_number":3},{"id":"e5e6529b30032f82382abcb624818a843996b2df","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:39:35.000000000","message":"Patch Set 3: Fails\n\nBuild Failed \n \nhttps://jenkins.openstack.org/job/nova-merge/117/ : SUCCESS \nhttps://jenkins.openstack.org/job/nova-pep8/1489/ : SUCCESS \nhttps://jenkins.openstack.org/job/nova/121639/ : FAILURE","accounts_in_message":[],"_revision_number":3},{"id":"e1c979b1bb249161a506a21c6ad9f7099c6c9a76","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2011-10-13 05:47:12.000000000","message":"Patch Set 3:\n\nTransient test failure... trying again.","accounts_in_message":[],"_revision_number":3},{"id":"840a514c28e18274700ef90c991f701ca4b90bd1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:47:45.000000000","message":"Patch Set 3: No score\n\nBuild Started https://jenkins.openstack.org/job/nova-merge/118/ (1/3)","accounts_in_message":[],"_revision_number":3},{"id":"4b25762f7ddf801b444f87ba7238acc24ef63c8b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:48:14.000000000","message":"Patch Set 3:\n\nBuild Started https://jenkins.openstack.org/job/nova-pep8/1490/ (2/3)","accounts_in_message":[],"_revision_number":3},{"id":"6e7b243d28f14f340c37d4154b05779ce1db74aa","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:48:58.000000000","message":"Patch Set 3:\n\nBuild Started https://jenkins.openstack.org/job/nova/121640/ (3/3)","accounts_in_message":[],"_revision_number":3},{"id":"049f128e1bf3f916549910b068660772da66ce43","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:58:39.000000000","message":"Patch Set 3: Verified\n\nBuild Successful \n \nhttps://jenkins.openstack.org/job/nova-merge/118/ : SUCCESS \nhttps://jenkins.openstack.org/job/nova-pep8/1490/ : SUCCESS \nhttps://jenkins.openstack.org/job/nova/121640/ : SUCCESS","accounts_in_message":[],"_revision_number":3},{"id":"53eba50d839acfafd925bc080d6d2db22e7870fc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-10-13 05:58:40.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"5ae7fb0a49a0072834b6a82938668cc8c962661f","revisions":{"06a27165dfd940b505df8c9677157b6bf70c0b2e":{"kind":"REWORK","_number":1,"created":"2011-09-27 15:03:29.000000000","uploader":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"ref":"refs/changes/84/684/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/684/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/684/1"}}},"commit":{"parents":[{"commit":"21da183edee9c8f8ddf0ff6a880486c0ed2e4623","subject":"Make sure options is set before checking managed_disk setting. Fixes bug 860520.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/21da183edee9c8f8ddf0ff6a880486c0ed2e4623"}]}],"author":{"name":"Brendan Maguire","email":"B_Maguire@Dell.com","date":"2011-09-16 13:56:13.000000000","tz":60},"committer":{"name":"Brendan Maguire","email":"B_Maguire@Dell.com","date":"2011-09-27 15:02:45.000000000","tz":60},"subject":"api.ec2.admin unit tests","message":"api.ec2.admin unit tests\n\nChange-Id: Ibc1fec34c7aee40c1252da8a3d174683434269c1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/06a27165dfd940b505df8c9677157b6bf70c0b2e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/06a27165dfd940b505df8c9677157b6bf70c0b2e"}]},"branch":"refs/heads/master"},"c5bf9c8ec455462cfd9cc73d67b65e94d6ae0fda":{"kind":"REWORK","_number":2,"created":"2011-09-28 09:06:57.000000000","uploader":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"ref":"refs/changes/84/684/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/684/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/684/2"}}},"commit":{"parents":[{"commit":"a2646129bc9dbd9dec57bdde7f510e0ea7bbddea","subject":"Merge \"compute_api create*() and schedulers refactoring\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a2646129bc9dbd9dec57bdde7f510e0ea7bbddea"}]}],"author":{"name":"Brendan Maguire","email":"B_Maguire@Dell.com","date":"2011-09-16 13:56:13.000000000","tz":60},"committer":{"name":"Brendan Maguire","email":"B_Maguire@Dell.com","date":"2011-09-28 09:06:54.000000000","tz":60},"subject":"api.ec2.admin unit tests","message":"api.ec2.admin unit tests\n\nChange-Id: Ibc1fec34c7aee40c1252da8a3d174683434269c1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c5bf9c8ec455462cfd9cc73d67b65e94d6ae0fda"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c5bf9c8ec455462cfd9cc73d67b65e94d6ae0fda"}]},"branch":"refs/heads/master"},"5ae7fb0a49a0072834b6a82938668cc8c962661f":{"kind":"REWORK","_number":3,"created":"2011-09-29 12:57:11.000000000","uploader":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"ref":"refs/changes/84/684/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/684/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/684/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/684/3"}}},"commit":{"parents":[{"commit":"ef22c0054ccb846dd7e81ba35f7e9c2b533d5ff7","subject":"Merge \"Fixes a test failure in master\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/ef22c0054ccb846dd7e81ba35f7e9c2b533d5ff7"}]}],"author":{"name":"Brendan Maguire","email":"B_Maguire@Dell.com","date":"2011-09-16 13:56:13.000000000","tz":60},"committer":{"name":"Brendan Maguire","email":"B_Maguire@Dell.com","date":"2011-09-29 12:57:08.000000000","tz":60},"subject":"api.ec2.admin unit tests","message":"api.ec2.admin unit tests\n\nChange-Id: Ibc1fec34c7aee40c1252da8a3d174683434269c1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5ae7fb0a49a0072834b6a82938668cc8c962661f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5ae7fb0a49a0072834b6a82938668cc8c962661f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}