ProphetStor driver does not honor create from snapshot volume size

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

Bug Description

create_volume_from_snapshot can specify a volume size larger than the orginal snapshot. Current code does not appear to handle this size difference.

Changed in cinder:
importance: Undecided → Low
wuyuting (wytdahu)
Changed in cinder:
assignee: nobody → wuyuting (wytdahu)
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/329741

Changed in cinder:
status: New → In Progress
Changed in cinder:
assignee: wuyuting (wytdahu) → Erlon R. Cruz (sombrafam)
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 Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/329741
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.

Changed in cinder:
assignee: Erlon R. Cruz (sombrafam) → Xinli Guan (xinli-v)
Changed in cinder:
assignee: Xinli Guan (xinli-v) → Rick Chen (rick-chen)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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

commit 985e008c17f6cc031c3a52cd2e5fc58d31ff77f6
Author: wuyuting <email address hidden>
Date: Tue Jun 14 22:12:16 2016 -0400

    ProphetStor driver create vol from snap with wrong size

    Create_volume_from_snapshot can specify a volume size larger than
    the original snapshot. ProphetStor driver not appear to handle this
    size difference. This patch fixes it.

    Co-Authored-By: Xinli Guan <email address hidden>

    Change-Id: I5d1dcd8f9bb892a949e562a58e71161a85bce891
    Closes-Bug: #1560661

Changed in cinder:
status: In Progress → Fix Released
Xinli Guan (xinli-v)
Changed in cinder:
importance: Low → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 11.0.0.0b1

This issue was fixed in the openstack/cinder 11.0.0.0b1 development milestone.

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

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.