]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix mandatory and optional args for create_volume
authorAvishay Traeger <avishay@il.ibm.com>
Wed, 25 Sep 2013 05:41:21 +0000 (08:41 +0300)
committerAvishay Traeger <avishay@il.ibm.com>
Wed, 25 Sep 2013 08:20:22 +0000 (11:20 +0300)
commitd534811cbcfe591688b5e9c6becbed2224be3c31
tree52b872fd406d1b9a402bc796384c7c4784173cc0
parentc77868f44b1e72be2b65f697a1f0f3b32126e581
Fix mandatory and optional args for create_volume

Calls to create_volume (both volume_rpcapi and scheduler_rpcapi) pass
mandatory args as optional and reverse. This patch fixes the calls to be
according to the function definition.

Also, fix the call to create_volume in the migration code where
allow_reschedule defaults to True, but we don't pass request_spec so it
prints a warning and disables it.

Change-Id: I9509f88866e0ae182f90ebdfca1fcf415ca60d19
Closes-Bug: #1230069
cinder/scheduler/filter_scheduler.py
cinder/tests/test_volume.py
cinder/volume/flows/create_volume/__init__.py
cinder/volume/manager.py