Bug 1341145 - Internal engine error when editing cluster if there is an external VM
Summary: Internal engine error when editing cluster if there is an external VM
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backend.Core
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ovirt-4.0.0-rc3
: 4.0.0
Assignee: Marek Libra
QA Contact: Nisim Simsolo
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-05-31 11:11 UTC by Tomas Jelinek
Modified: 2016-08-25 12:43 UTC (History)
4 users (show)

Fixed In Version: 4.0.0.5
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-08-25 12:43:05 UTC
oVirt Team: Virt
Embargoed:
rule-engine: ovirt-4.0.0+
rule-engine: planning_ack+
rule-engine: devel_ack+
mavital: testing_ack+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 58365 0 master MERGED core: No UpdateVmCommand after Cluster Edit for external VMs 2020-04-14 08:13:45 UTC
oVirt gerrit 59049 0 ovirt-engine-4.0 MERGED core: No UpdateVmCommand after Cluster Edit for external VMs 2020-04-14 08:13:45 UTC

Description Tomas Jelinek 2016-05-31 11:11:54 UTC 
Steps:
1: have an env with external Vms (at least one)
2: edit the cluster

Result:
Internal engine error is reported

Expected result:
Some descriptive validation error is reported that can not edit the cluster if there are external VMs in it.
Or it could ignore the external VMs and not try to update tham

Additional info:
Fails because the UpdateVmCommand.validate() returns false making the UpdateClusterCommand call setSucceeded(false) in because !updateVms()
Comment 2 Nisim Simsolo 2016-08-22 14:14:01 UTC 
Verified: 
ovirt-engine-4.0.2.7-0.1.el7ev.noarch
qemu-kvm-rhev-2.3.0-31.el7_2.21.x86_64
vdsm-4.18.11-1.el7ev.x86_64
libvirt-client-1.2.17-13.el7_2.5.x86_64
sanlock-3.2.4-3.el7_2.x86_64
Note You need to log in before you can comment on or make changes to this bug.