]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
VMware: Add missing storage profile requirement
authorVipin Balachandran <vbala@vmware.com>
Thu, 27 Nov 2014 14:55:42 +0000 (20:25 +0530)
committerVipin Balachandran <vbala@vmware.com>
Wed, 3 Dec 2014 07:21:10 +0000 (12:51 +0530)
commitb8b3590dbd4554c5d49d54252e52c8ec448a8464
treed29c318d759e476c8c9d363ef03486d5bf03495b
parent530c17810d52b056c2f56145f78d57011bfe8901
VMware: Add missing storage profile requirement

If a storage profile is part of a volume type, it should be used as a
requirement for datastore selection-- only datastores which satisfy the
storage profile should be used for backing VM creation. Currently in
retype API, if storage profiles in old and new volume types are same,
it is not passed as a requirement. Hence, the backing VM corresponding
to the volume might end up in a datastore which doesn't satisfy the
storage profile. This patch fixes the problem.

Closes-Bug: #1398293
Change-Id: I49161e9fc5a8f2749ee6097fa5a136b78dfcf3ab
cinder/tests/test_vmware_vmdk.py
cinder/volume/drivers/vmware/vmdk.py