Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 872930

Summary: engine [Live Storage Migration]: we can create vm from snapshot during storage live migration
Product: Red Hat Enterprise Virtualization Manager Reporter: Dafna Ron <dron>
Component: ovirt-engineAssignee: Daniel Erez <derez>
Status: CLOSED CURRENTRELEASE QA Contact: Dafna Ron <dron>
Severity: high Docs Contact:
Priority: high    
Version: 3.1.0CC: abaron, acathrow, amureini, dyasny, hateya, iheim, lpeer, Rhev-m-bugs, vvyazmin, yeylon, ykaul
Target Milestone: ---   
Target Release: 3.1.0   
Hardware: x86_64   
OS: Linux   
Whiteboard: storage
Fixed In Version: SI24.1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-04 19:58:31 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-11-04 12:12:53 UTC
Created attachment 637985 [details]
log

Description of problem:

one the migration snapshot is created we no longer lock it so we are able to create a vm from the snapshot while the image migration is running. 

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

si23

How reproducible:

100%

Steps to Reproduce:
1. create and run a vm
2. move disk
3. after the migration snapshot is created clone a vm
  
Actual results:

Task succeeds and currently we have NPE on clear task which was already solved for next build.

Expected results:

I think that creation a clone from the migrate snapshot as long as the disk image is still locked might cause image corruption and should not be allowed as long as the live migration is running. 

Additional info: log

Comment 1 Daniel Erez 2012-11-05 12:36:35 UTC
Patch sent upstream:
http://gerrit.ovirt.org/#/c/9031/
Change-Id: I9899744859475b2d27ca858c21279fb439768f9b

Comment 2 Allon Mureinik 2012-11-06 15:57:52 UTC
Merged I9899744859475b2d27ca858c21279fb439768f9b

Comment 3 Dafna Ron 2012-11-12 10:03:08 UTC
changed fixed in version to si24.1 after speaking to Ayal
verified on si24.1 
there is a CanDoAction preventing the clone