Need elevated context when accepting transfer

Bug #1187538 reported by John Griffith on 2013-06-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
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.

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

Changed in cinder:
status: Triaged → In Progress

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) on 2013-07-17
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-2 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers