Bug 838944

Summary: ovirt-engine-backend: when tasks fail on snapshot we change the snapshot status to BROKEN
Product: Red Hat Enterprise Virtualization Manager Reporter: Dafna Ron <dron>
Component: ovirt-engineAssignee: Ayal Baron <abaron>
Status: CLOSED WONTFIX QA Contact: Haim <hateya>
Severity: high Docs Contact:
Priority: high    
Version: 3.1.0CC: abaron, amureini, dyasny, hateya, iheim, lpeer, Rhev-m-bugs, yeylon, ykaul
Target Milestone: ---   
Target Release: 3.1.0   
Hardware: x86_64   
OS: Linux   
Whiteboard: storage
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-27 13:37:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
log none

Description Dafna Ron 2012-07-10 13:33:30 UTC
Created attachment 597337 [details]
log

Description of problem:

I failed a delete snapshot task in the spm  host and snapshot is getting status BROKEN. 
1. even if we decide that the snapshot is no longer usable, BROKEN is not a good way to describe it (I am assuming that this was decided because engine assumes that the chain is broken) 
2. vdsm can rollback on some actions and even delete if not yet sent to storage so just because the task failed does not mean that engine should decide that the chain was broken and that the image is no longer usable. 

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

si9.1

How reproducible:

100%

Steps to Reproduce:
1. create a vm with large disk -> create snapshots
2. delete a snapshot
3. use vdsClinet -s 0 stopTask <task id> to stop the task
  
Actual results:

snapshot status will change to BROKEN

Expected results:

1. BROKEN is not a good way to phrase it (even if snapshot is no longer usable)
2. engine has no idea what is the image status so why decide that it is no longer usable and that the chain is broken?
3. in vdsm, if the delete has started there is no way of rolling back and this may cause a snapshot to still exist in db but not in storage. in which case, engine will be the one causing the chain to be broken instead of removing item from db. 

Additional info: logs

Comment 2 Itamar Heim 2013-01-27 13:37:30 UTC
Closing old bugs. If this issue is still relevant/important in current version, please re-open the bug.