)]}'
{"id":"openstack%2Fnova~17072","triplet_id":"openstack%2Fnova~stable%2Ffolsom~I3d47b5a48eae21ea913120bd00728a0b8132da81","project":"openstack/nova","branch":"stable/folsom","topic":"bug/1074437","hashtags":[],"change_id":"I3d47b5a48eae21ea913120bd00728a0b8132da81","subject":"Use elevated context in disassociate_floating_ip","status":"MERGED","created":"2012-11-28 19:24:21.000000000","updated":"2012-11-29 00:21:38.000000000","submitted":"2012-11-29 00:21:36.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f92c153d460802a75889df8e631c4413a12c5bd5","_number":17072,"virtual_id_number":17072,"owner":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2012-11-29 00:21:36.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"value":2,"date":"2012-11-28 23:43:21.000000000","_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":{"approved":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2012-11-28 19:24:21.000000000","_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"value":1,"date":"2012-11-28 23:43:21.000000000","_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":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":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-11-28 19:24:21.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":"2012-11-28 23:43:21.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":"2012-11-29 00:21:36.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"4e6eeb27de1d1d5ab6833b0d5b43c46197129b03","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-28 19:44:56.000000000","message":"Patch Set 1: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/17072/1/check/gate-nova-merge/13112 : SUCCESS\n- http://logs.openstack.org/17072/1/check/gate-nova-docs/8836 : SUCCESS\n- http://logs.openstack.org/17072/1/check/gate-nova-pep8/14146 : SUCCESS\n- http://logs.openstack.org/17072/1/check/gate-nova-python26/10286 : SUCCESS\n- http://logs.openstack.org/17072/1/check/gate-nova-python27/10356 : SUCCESS\n- http://logs.openstack.org/17072/1/check/gate-tempest-devstack-vm/20330 : SUCCESS\n- http://logs.openstack.org/17072/1/check/gate-tempest-devstack-vm-cinder/5671 : SUCCESS\n- http://logs.openstack.org/17072/1/check/gate-nova-pylint/3667 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"ff17a1d9ede1d822923c07da689605c890e423c3","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2012-11-28 23:43:21.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\nOk, this is a very serious bug and the fix is quite trivial and safe. Fix has been on master for nearly a month without any known issues.\n\nApproving as an exception to the 2012.2.1 freeze.","accounts_in_message":[],"_revision_number":1},{"id":"b11a2954b47682912f0b06f578849baf3cf9a4a2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-28 23:43:23.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttps://jenkins.openstack.org/zuul/status","accounts_in_message":[],"_revision_number":1},{"id":"4faa14de2e62081fcc8975f60fdfafbde8dbeb4f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-29 00:21:36.000000000","message":"Patch Set 1: Verified\n\nBuild successful\n\n- http://logs.openstack.org/17072/1/gate/gate-nova-merge/13142 : SUCCESS\n- http://logs.openstack.org/17072/1/gate/gate-nova-docs/8866 : SUCCESS\n- http://logs.openstack.org/17072/1/gate/gate-nova-pep8/14176 : SUCCESS\n- http://logs.openstack.org/17072/1/gate/gate-nova-python26/10316 : SUCCESS\n- http://logs.openstack.org/17072/1/gate/gate-nova-python27/10386 : SUCCESS\n- http://logs.openstack.org/17072/1/gate/gate-tempest-devstack-vm/20373 : SUCCESS\n- http://logs.openstack.org/17072/1/gate/gate-tempest-devstack-vm-cinder/5673 : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"5a6f48754b9fb1dbc6c7f988d9f9e13dc90eb0bb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-29 00:21:38.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"3ede373f281edb07141b7e8752670c16878eb909","revisions":{"3ede373f281edb07141b7e8752670c16878eb909":{"kind":"REWORK","_number":1,"created":"2012-11-28 19:24:21.000000000","uploader":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"ref":"refs/changes/72/17072/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/72/17072/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/72/17072/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/72/17072/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/72/17072/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/72/17072/1"}}},"commit":{"parents":[{"commit":"e7877868b13ff615feb61a2ac229b160d1f51283","subject":"Improve provider_location cleanup code for RBD.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e7877868b13ff615feb61a2ac229b160d1f51283"}]}],"author":{"name":"Brian Waldon","email":"bcwaldon@gmail.com","date":"2012-11-02 17:47:34.000000000","tz":-420},"committer":{"name":"Vishvananda Ishaya","email":"vishvananda@gmail.com","date":"2012-11-28 19:24:07.000000000","tz":-480},"subject":"Use elevated context in disassociate_floating_ip","message":"Use elevated context in disassociate_floating_ip\n\nThe call to service_get_by_host_and_topic would raise AdminRequired\nin the case that a non-admin tried to disassociate a floating ip\nfrom their instance and multi_host was on. Using an elevated\ncontext for the db lookup enables the host lookup to happend and\nallows a user to disassociate their floating ips.\n\nFixes bug 1074437\n\nChange-Id: I3d47b5a48eae21ea913120bd00728a0b8132da81\n(cherry picked from commit eedd98477dd3b8a5561d30732b7ccf71e9a7d428)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3ede373f281edb07141b7e8752670c16878eb909"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3ede373f281edb07141b7e8752670c16878eb909"}]},"branch":"refs/heads/stable/folsom"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
