]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
coraid: fix snapshot deletion
authorRoman Bogorodskiy <rbogorodskiy@mirantis.com>
Thu, 4 Sep 2014 10:04:59 +0000 (10:04 +0000)
committerRoman Bogorodskiy <rbogorodskiy@mirantis.com>
Fri, 12 Sep 2014 17:04:33 +0000 (17:04 +0000)
commitb39446b46b83af2dcf36f091ed9dcaf4d29410cf
tree7efc6a5710898e4d3fef8f7bbf1928ffe51f3880
parenteb01ccba28b0feabe6220c125776fee3c2c52505
coraid: fix snapshot deletion

Currently, a request to ESM API for a snapshot deletion looks this way:

{'addr': 'cms',
 'data': {
    'repoName': repository_name,
    'lvName': snapshot_name},
 'op': 'orchStrLunMods',
 'args': 'delClSnap'}

This is correct, however, some versions of the ESM API have a bug that
prevents it from validation this valid request and it fails with:

Must define the new LV name to be assigned to the clone/clSnap.

As a workaround for this bug, provide an 'newLvName' property with a
'noop' value to make the validation pass even on buggy versions.

Fixes bug #1365417

Change-Id: I3fd7f3c6a64cac992735b098368b52b9ab21d85c
cinder/tests/test_coraid.py
cinder/volume/drivers/coraid.py