]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Fixed a volume creation re-schedule error
authorLiang Chen <cbjchen@cn.ibm.com>
Fri, 5 Apr 2013 14:00:39 +0000 (22:00 +0800)
committerLiang Chen <cbjchen@cn.ibm.com>
Fri, 5 Apr 2013 14:00:39 +0000 (22:00 +0800)
Don't see any reason to construct a list for request_spec['volume_id'].
And it indeed causes problems when the scheduler receives a reschedule
request with such a request_spec.

Fixes bug #1164857

Change-Id: I811e957d5bf9c814165d55ea0042833ea72c8de5

cinder/volume/manager.py

index 8a90b0c2567900aa02aa2b6e45171076ca312481..16741eb2f8cad882859f5a02bf3496245182a178 100644 (file)
@@ -368,7 +368,7 @@ class VolumeManager(manager.SchedulerDependentManager):
             LOG.debug(_("No request spec, will not reschedule"))
             return
 
-        request_spec['volume_id'] = [volume_id]
+        request_spec['volume_id'] = volume_id
 
         LOG.debug(_("volume %(volume_id)s: re-scheduling %(method)s "
                     "attempt %(num)d") %