Bug 1297842 - [cinder] revert operation when deleting Cinder snapshot should not update the active flag of the snapshot's volume
Summary: [cinder] revert operation when deleting Cinder snapshot should not update the...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ovirt-3.6.2
: 3.6.2.6
Assignee: Maor
QA Contact: Natalie Gavrielov
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-12 15:34 UTC by Maor
Modified: 2016-02-23 13:32 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-02-23 13:32:06 UTC
oVirt Team: Storage
Embargoed:
amureini: ovirt-3.6.z?
ylavi: planning_ack+
amureini: devel_ack+
rule-engine: testing_ack?


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 51727 0 master MERGED core: delete Cinder snapshot failover. 2016-01-13 07:10:21 UTC
oVirt gerrit 51747 0 ovirt-engine-3.6 MERGED core: delete Cinder snapshot failover. 2016-01-13 13:03:42 UTC
oVirt gerrit 52353 0 ovirt-engine-3.6.2 MERGED core: delete Cinder snapshot failover. 2016-01-18 18:03:40 UTC

Description Maor 2016-01-12 15:34:26 UTC
Description of problem:
The revert operation of delete Cinder snapshot (The OnFailed), should not change the active flag of the disk.

Version-Release number of selected component (if applicable):


How reproducible:
100%

Steps to Reproduce:
1. Create a VM with Cinder disk
2. Create a snapshot
3. Create another snapshot
4. delete the last snapshot and fail the operation of the delete

Actual results:
The first snapshot will become active and we will have two volumes which are active in the DB

Expected results:
We should not update the active volume

Additional info:

Comment 1 Maor 2016-01-12 17:48:06 UTC
Please consider that for this fix to work on multiple disks the patch with the following change-id I2ca19ec44cba371813381e8f8b076e463eda4173 should be backported.

Until then, this fix should resolve the issue but mainly for one Cinder disk, for multiple disks there might be several corner cases which might not be supported.

Comment 2 Red Hat Bugzilla Rules Engine 2016-01-13 11:27:47 UTC
Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.


Note You need to log in before you can comment on or make changes to this bug.