commit 26aa01094a79939320d58f2fe2d5731f169987b1
Author: Chris Behrens <email address hidden>
Date: Wed Mar 20 18:13:05 2013 +0000
Change arguments to volume_detach()
Most compute API calls accept an instance object. To be compatible with
cells, volume_detach() should as well. Also change it to accept the
volume object, instead of just a volume ID. This lets us proxy the
volume object to child cells a bit more easily.
This removes 2 skipTests that we have related to volumes and cells.
There was also a check that if instance_get_by_uuid() returned no
instance, it VolumeUnattached should be raised. However, this check
would never work as InstanceNotFound would have been raised. This
patch also corrects that.
Reviewed: https:/ /review. openstack. org/24921 github. com/openstack/ nova/commit/ 26aa01094a79939 320d58f2fe2d573 1f169987b1
Committed: http://
Submitter: Jenkins
Branch: master
commit 26aa01094a79939 320d58f2fe2d573 1f169987b1
Author: Chris Behrens <email address hidden>
Date: Wed Mar 20 18:13:05 2013 +0000
Change arguments to volume_detach()
Most compute API calls accept an instance object. To be compatible with
cells, volume_detach() should as well. Also change it to accept the
volume object, instead of just a volume ID. This lets us proxy the
volume object to child cells a bit more easily.
This removes 2 skipTests that we have related to volumes and cells.
Fixes bug 1157408
There was also a check that if instance_ get_by_ uuid() returned no
instance, it VolumeUnattached should be raised. However, this check
would never work as InstanceNotFound would have been raised. This
patch also corrects that.
Change-Id: Ide35ad5d8e147f 6bd21b5840b9013 63dab0eae38