Bug 1364137 - make VM template should be blocked while importing this VM.
Summary: make VM template should be blocked while importing this VM.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Virt
Version: 4.0.2.2
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ovirt-4.1.1
: 4.1.1
Assignee: Sharon Gratch
QA Contact: Nisim Simsolo
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-04 14:19 UTC by Nisim Simsolo
Modified: 2017-04-21 09:43 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-04-21 09:43:53 UTC
oVirt Team: Virt
Embargoed:
rule-engine: ovirt-4.1+
rule-engine: planning_ack+
rule-engine: devel_ack+
mavital: testing_ack+


Attachments (Terms of Use)
Video capture of this issue (3.75 MB, video/mp4)
2016-08-04 14:20 UTC, Nisim Simsolo
no flags Details
vdsm.log.gz (1.21 MB, application/x-gzip)
2016-08-04 14:25 UTC, Nisim Simsolo
no flags Details
engine.log.gz (894.45 KB, application/x-gzip)
2016-08-04 14:26 UTC, Nisim Simsolo
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 71008 0 master MERGED backend: add "shared" lock for the AddVmTemplateCommand 2017-01-29 12:26:49 UTC
oVirt gerrit 71473 0 master MERGED webadmin: update actions availability of VMs list with "exclusive" lock 2017-02-06 16:35:38 UTC
oVirt gerrit 71478 0 ovirt-engine-4.1 MERGED backend: add "shared" lock for the AddVmTemplateCommand 2017-02-01 12:39:14 UTC
oVirt gerrit 71750 0 ovirt-engine-4.1 MERGED webadmin: update actions availability of VMs list with "exclusive" lock 2017-02-07 10:15:19 UTC

Description Nisim Simsolo 2016-08-04 14:19:13 UTC
Description of problem:
When importing VM from an external provider, during "locked" VM, before copying disks phase, it is possible to make a template from that VM.
This option should be blocked when VM is locked (when import is in copying disks phase, this option changed to grayed out).
VM created from disk template will run with a corrupted disk image.

Version-Release number of selected component (if applicable):
ovirt-engine-4.0.2.2-0.1.el7ev
vdsm-4.18.9-1.el7ev.x86_64
qemu-kvm-rhev-2.3.0-31.el7_2.20.x86_64
libvirt-client-1.2.17-13.el7_2.5.x86_64
sanlck-3.2.4-3.el7_2.x86_64

How reproducible:
Consistently

Steps to Reproduce:
1. import VM from an external provider.
2. Wait for padlock icon to appear on VM and make a template from this VM.
3.

Actual results:
Template is being created.
VM created from this template failed to load because of missing boot parameters.

Expected results:
Make template option should be blocked during the import process.

Additional info:
Video capture of this issue attached.
engine and vdsm logs attached.

Comment 1 Nisim Simsolo 2016-08-04 14:20:38 UTC
Created attachment 1187525 [details]
Video capture of this issue

Comment 2 Nisim Simsolo 2016-08-04 14:25:46 UTC
Created attachment 1187529 [details]
vdsm.log.gz

Comment 3 Nisim Simsolo 2016-08-04 14:26:35 UTC
Created attachment 1187537 [details]
engine.log.gz

Comment 4 Tomas Jelinek 2016-08-05 05:58:33 UTC
Should be enough to add a validate() and updateActionAvailability() for VMs which are locked. Fix should be trivial, targeting to 4.1 and if it will all go well we may backport.

Comment 5 Nisim Simsolo 2017-02-12 15:05:56 UTC
Verification builds:
ovirt-engine-4.1.1-0.1.el7
libvirt-client-2.0.0-10.el7_3.4.x86_64
vdsm-4.19.4-1.el7ev.x86_64
qemu-kvm-rhev-2.6.0-28.el7_3.3.x86_64
sanlock-3.4.0-1.el7.x86_64
virt-v2v-1.32.7-3.el7_3.2.x86_64


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