Bug 1398287

Summary: Live migration to different cluster doesn't work for RHV
Product: Red Hat CloudForms Management Engine Reporter: movciari
Component: ProvidersAssignee: Martin Betak <mbetak>
Status: CLOSED CURRENTRELEASE QA Contact: Angelina Vasileva <anikifor>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 5.7.0CC: cpelland, istein, jfrey, jhardy, mbetak, obarenbo, oourfali, simaishi, tjelinek
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1416077 (view as bug list) Environment:
Last Closed: 2017-06-12 16:27:31 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: RHEVM Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1416077    
Attachments:
Description Flags
engine.log
none
rhevm.log
none
evm.log
none
production.log
none
automation.log none

Description movciari 2016-11-24 11:49:03 UTC
Created attachment 1223813 [details]
engine.log

Description of problem:
When trying to live migrate RHV VM to host in a different cluster using CFME, it fails.
When doing same thing from RHV webadmin, it works

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

How reproducible:
always

Steps to Reproduce:
1. Perform live migration of VM to different cluster
2. Wait for migration to complete

Actual results:
It fails

Expected results:
Migration should work through CFME if it works through RHV webadmin

Additional info:

Comment 2 movciari 2016-11-24 11:49:27 UTC
Created attachment 1223814 [details]
rhevm.log

Comment 3 movciari 2016-11-24 11:50:11 UTC
Created attachment 1223815 [details]
evm.log

Comment 4 movciari 2016-11-24 11:51:47 UTC
Created attachment 1223816 [details]
production.log

Comment 5 movciari 2016-11-24 11:52:20 UTC
Created attachment 1223817 [details]
automation.log

Comment 6 Tomas Jelinek 2016-12-12 14:09:20 UTC
https://github.com/ManageIQ/manageiq/pull/13077

Removed the option from miq to pick a cluster since this is not supposed to be supported.

Comment 10 CFME Bot 2017-01-23 19:51:13 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/2fbf505a904bf56bb707958ee717e32c2f7310cd

commit 2fbf505a904bf56bb707958ee717e32c2f7310cd
Author:     Tomas Jelinek <tjelinek>
AuthorDate: Fri Jan 6 04:28:31 2017 -0500
Commit:     Martin Betak <mbetak>
CommitDate: Mon Jan 23 17:55:16 2017 +0100

    rhv: removed the option to migrate the VMs outside of the cluster.
    
    The support for cross cluster migrations was added to oVirt only as a
    workaround for el6->el7 migrations but should not be exposed. Since the current
    code in manageiq anyway did not work properly, removing the support for it
    completely - it is a low level functionality, it is obsoleted and discouraged
    to be used.
    
    Links
    https://bugzilla.redhat.com/show_bug.cgi?id=1398287

 app/models/manageiq/providers/redhat/infra_manager.rb |  9 ++++++++-
 app/models/vm_migrate_workflow.rb                     | 18 ++++++++++++++----
 spec/models/vm_migrate_workflow_spec.rb               |  4 ++--
 3 files changed, 24 insertions(+), 7 deletions(-)

Comment 11 CFME Bot 2017-01-24 10:17:54 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/926812bcd0a0e9f54e02f8e45dd0654a281caacb

commit 926812bcd0a0e9f54e02f8e45dd0654a281caacb
Author:     Tomas Jelinek <tjelinek>
AuthorDate: Fri Jan 6 04:32:59 2017 -0500
Commit:     Martin Betak <mbetak>
CommitDate: Tue Jan 24 11:03:46 2017 +0100

    rhv: removed the option to migrate the VMs outside of the cluster.
    
    The support for cross cluster migrations was added to oVirt only as a
    workaround for el6->el7 migrations but should not be exposed. Since the current
    code in manageiq anyway did not work properly, removing the support for it
    completely - it is a low level functionality, it is obsoleted and discouraged
    to be used.
    
    Links
    https://bugzilla.redhat.com/show_bug.cgi?id=1398287

 app/controllers/application_controller.rb               | 11 +++++++++++
 app/views/miq_request/_prov_vm_migrate_dialog.html.haml | 13 +++++++------
 2 files changed, 18 insertions(+), 6 deletions(-)

Comment 12 Oved Ourfali 2017-01-24 10:29:58 UTC
Tomas, you'll need to have a new PR for EUWE, as the repo split means the backport will not work.
Can you backport?