Need elevated context when accepting transfer

Bug #1187538 reported by John Griffith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
John Griffith

Bug Description

The transfer_accept method in transfer/api.py is trying to access the volume reservations from the donor before the ownership has been transferred. In other words, we call accept as the "new" owner and try to adjust quotas for the "old" owners quotas upon transfer.

Simply fix by elevating the context on the quota reservation update call.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/31690

Changed in cinder:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/31690
Committed: http://github.com/openstack/cinder/commit/e02cf9c373daa7b03e141e016fb166841f6b02b7
Submitter: Jenkins
Branch: master

commit e02cf9c373daa7b03e141e016fb166841f6b02b7
Author: John Griffith <email address hidden>
Date: Tue Jun 4 12:59:20 2013 -0600

    Elevate acceptors context on accept reserve udpate

    When accepting a transfer we need to update the
    quota data for both the donor and the acceptor's.
    The acceptor is easy enough (as that's the context
    we currently are using) however to update the donor's
    quota data we need an elevated context.

    Fixes bug: 1187538

    Change-Id: I1fb9e7726830dce2c3de23d3446de5bbe9a1fd24

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-2 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.