Bug 1383738 - Legacy migration settings don't apply
Summary: Legacy migration settings don't apply
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backend.Core
Version: 4.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ovirt-4.0.5
: 4.0.5.2
Assignee: Tomas Jelinek
QA Contact: Israel Pinto
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-11 15:59 UTC by Milan Zamazal
Modified: 2017-01-18 07:38 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-01-18 07:38:43 UTC
oVirt Team: Virt
Embargoed:
tjelinek: ovirt-4.0.z?
rule-engine: planning_ack?
tjelinek: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 65381 0 master MERGED core: autoconverge and compression taken from legacy policy 2020-08-11 05:52:32 UTC
oVirt gerrit 65383 0 ovirt-engine-4.0 MERGED core: autoconverge and compression taken from legacy policy 2020-08-11 05:52:32 UTC
oVirt gerrit 65384 0 ovirt-engine-4.0.5 MERGED core: autoconverge and compression taken from legacy policy 2020-08-11 05:52:32 UTC

Description Milan Zamazal 2016-10-11 15:59:23 UTC
Description of problem:

When legacy migration is selected, autoconverge and compression migration settings as set by the user for the cluster or the particular VM are ignored and their values sent to Vdsm are always false.

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

Current master.

How reproducible:

Always.

Steps to Reproduce:
1. Open Edit Cluster dialog.
2. Select "Legacy" migration policy.
3. Select "Auto Converge" and "Compress" in Additional Properties.
4. Migrate a VM in the cluster.
5. Check Vdsm log (DEBUG level needed) for the migration parameters.

Actual results:

"autoConverge" and "compress" parameter values are "false".

Expected results:

"autoConverge" and "compress" parameter values are "true" as requested for the cluster.

Additional info:

Comment 1 Tomas Jelinek 2016-10-12 08:34:32 UTC
did not get into build, moving back to post

Comment 2 Michal Skrivanek 2016-10-12 09:26:34 UTC
all merged

Comment 3 Michal Skrivanek 2016-10-14 08:56:46 UTC
I still wonder how this hook works but it should check the build creation/tag time and not when the build is finished, otherwise bugs like this - modified during those 1-2 days when the build is ongoing - are changed incorrectly

Comment 4 Israel Pinto 2016-10-26 07:11:43 UTC
Verify with:
Engine:Red Hat Virtualization Manager Version: 4.0.5.2-0.2.el7ev
Host:
OS Version:RHEL - 7.3 - 6.el7
OS Description:Red Hat Enterprise Linux Server 7.3 Beta (Maipo)
Kernel Version:3.10.0 - 510.el7.x86_64
KVM Version:2.6.0 - 27.el7
LIBVIRT Version:libvirt-2.0.0-10.el7
VDSM Version:vdsm-4.18.13-1.el7ev
SPICE Version:0.12.4 - 19.el7

Steps to Reproduce:
1. Open Edit Cluster dialog.
2. Select "Legacy" migration policy.
3. Select "Auto Converge" and "Compress" in Additional Properties.
   (On vm those options are inherit from cluster) 
4. Migrate a VM in the cluster.
5. Check Vdsm log (DEBUG level needed) for the migration parameters.

Results: both "Auto Converge" and "Compress" are enabled. 
From the log:
jsonrpc.Executor/3::DEBUG::2016-10-26 09:56:40,040::__init__::530::jsonrpc.JsonRpcServer::(_handle_request) Calling 'VM.migrate' in bridge with {'params': {'tunneled': 'false', 'dstqemu': '10.35.161.161', 'autoConverge': 'true', 'src': 'seal08.qa.lab.tlv.redhat.com', 'enableGuestEvents': False, 'dst': 'virt-nested-vm14.scl.lab.tlv.redhat.com:54321', 'vmId': '6adec1d9-519a-4ca7-b3b6-bae6369777a8', 'abortOnError': 'true', 'compressed': 'true', 'method': 'online'}, 'vmID': '6adec1d9-519a-4ca7-b3b6-bae6369777a8'}
jsonrpc.Executor/3::DEBUG::2016-10-26 09:56:40,041::API::540::vds::(migrate) {'tunneled': 'false', 'dstqemu': '10.35.161.161', 'autoConverge': 'true', 'src': 'seal08.qa.lab.tlv.redhat.com', 'enableGuestEvents': False, 'dst': 'virt-nested-vm14.scl.lab.tlv.redhat.com:54321', 'vmId': '6adec1d9-519a-4ca7-b3b6-bae6369777a8', 'abortOnError': 'true', 'compressed': 'true', 'method': 'online'}


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