]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix ownership transfer when cloning with SolidFire
authorMathieu Gagné <mgagne@iweb.com>
Thu, 23 May 2013 19:26:44 +0000 (15:26 -0400)
committerMathieu Gagné <mgagne@iweb.com>
Thu, 23 May 2013 19:37:39 +0000 (15:37 -0400)
commitaf023fe0cce3b8ef2b90ec37f1bc49feb17eac83
tree9b938da7527e06594ad88107bcbf8cbe239ce51a
parent561520447de9b89db7f052e31fc9d5f7d416b5dc
Fix ownership transfer when cloning with SolidFire

When cloning a volume with SolidFire driver, the owner of
the cloned volume is not set correctly in SolidFire when there is
a transfer of ownership.

This results in inconsistent states where the cloned volume
is owned by the new tenant in Cinder but SolidFire thinks it is still
owned by the original volume's tenant.

This patch adds the newAccountID parameter to all CloneVolume calls.

If the cloned volume is owned by the same tenant, newAccountID will
be set to the same value as the original SolidFire volume. There
will be no change of ownership done by the cloning process in SolidFire.

If the cloned volume should be owned by a different tenant, newAccountID
will be set to the appropriate SolidFire account corresponding
to the new tenant. If the SolidFire account does not exist already,
it will be created.

Fixes: bug #1183521
Change-Id: I622ca2962478298e3e0c5a26866e39919805075f
cinder/volume/drivers/solidfire.py