clone on busy SolidFire cluster may result in delay for _get_model_info

Bug #1129810 reported by John Griffith
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
John Griffith
Folsom
Won't Fix
High
John Griffith

Bug Description

Currently the _get_model_update method in the SolidFire driver tries to gather the volume info immediately after a clone operation is requested. Depending on the size of the volume and how busy the cluster is this operation may actually take a few seconds to complete. As things are currently, we do the check, if nothings returned we just return and empty model_upate.

To address this, add a poll routine for up to a certain number of seconds and raise/report an error if the clone isn't available within the specified period of time.

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/22395

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/22395
Committed: http://github.com/openstack/cinder/commit/737468e5c80a35b859dcda29706c88e2976339a3
Submitter: Jenkins
Branch: master

commit 737468e5c80a35b859dcda29706c88e2976339a3
Author: John Griffith <email address hidden>
Date: Tue Feb 19 21:34:01 2013 -0700

    Handle maxclonepervolume/node limits in SF driver.

    The SolidFire cluster has a max simultaneous clones per volume and node
    limt that we weren't accounting for. Add some logic to detect this situation
    and loop until enough simultaneous clones are completed to allow processing
    of the next one in line.

    Fixes bug: 1129810

    Change-Id: Id63c1b142a81df5d9d2c4e39fb821b03ebac63a2

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → grizzly-3
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/folsom)

Fix proposed to branch: stable/folsom
Review: https://review.openstack.org/25964

tags: added: folsom-backport-potential
Thierry Carrez (ttx)
Changed in cinder:
milestone: grizzly-3 → 2013.1
Mark McLoughlin (markmc)
tags: removed: folsom-backport-potential
Revision history for this message
Alan Pevec (apevec) wrote :

stable/folsom review was 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.