Violin driver does not honor clone volume size

Bug #1554787 reported by Sean McGinnis
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
zhangguoqing

Bug Description

It's possible to create a volume from a source volume with a different size. The create_cloned_volume call into the driver could have a larger volume['size'] than the source volume. So the driver will need to create the clone, then check if it needs to extend the volume to a new size.

Changed in cinder:
milestone: none → newton-1
Changed in cinder:
assignee: nobody → zhangguoqing (474751729-o)
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/295094

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

Reviewed: https://review.openstack.org/295094
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=f01946651acf10089352ce7f72c4affb0e201aa4
Submitter: Jenkins
Branch: master

commit f01946651acf10089352ce7f72c4affb0e201aa4
Author: zhangguoqing <zhang.guoqing@99cloud.net>
Date: Mon Mar 21 15:04:33 2016 +0800

    Violin: resize volume if cloned lun is larger

    1. In case the new volume is larger than the original one,
       resize the cloned volume.
    2. Add unit test by ddt.

    Closes-Bug: #1554787
    Change-Id: Iacd9ed354ac927666270cb717722bd14a34ffecd
    Depends-on: I829c0e6f99fff0ec0bd5366c9d2e79b1b073f0a3

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/cinder 9.0.0.0b2

This issue was fixed in the openstack/cinder 9.0.0.0b2 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/336092
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Jay Bryant (<email address hidden>) on branch: master
Review: https://review.openstack.org/336092
Reason: It was decided that we wouldn't go down this path so we this patch can be abandoned.

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.