On Thu, Nov 20, 2014 at 07:51:40AM +0000, Arne Wiebalck wrote:
 
 On 19 Nov 2014, at 18:36, Kashyap Chamarthy <kchamart(a)redhat.com> wrote:
 
 > [. . .]
 > 
 >> From what I understand, retype needs to be implemented on a per
 >> driver level, and that’s simply not the case for the rbd driver
 >> yet.
 >> 
 >> Below is a small patch to the rbd driver that “implements” the
 >> retype function and subsequently allows for QoS changes of Cinder
 >> volumes on Ceph.
 >> 
 >> (Not sure at all if that breaks things elsewhere :)
 > 
 > If you have time, you might want to submit an upstream review
 > request and let the CI infrastructure handle that for you.
 
 Done: 
https://review.openstack.org/#/c/135874/ 
Nice. It needs a respin -- PEP-8 indentation errors, commented in the
review.
 
-- 
/kashyap