Bug 1398287 - Live migration to different cluster doesn't work for RHV
Summary: Live migration to different cluster doesn't work for RHV
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: GA
: 5.8.0
Assignee: Martin Betak
QA Contact: Angelina Vasileva
URL:
Whiteboard:
Depends On:
Blocks: 1416077
TreeView+ depends on / blocked
 
Reported: 2016-11-24 11:49 UTC by movciari
Modified: 2019-05-28 08:55 UTC (History)
9 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1416077 (view as bug list)
Environment:
Last Closed: 2017-06-12 16:27:31 UTC
Category: Bug
Cloudforms Team: RHEVM
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
engine.log (14.41 KB, text/plain)
2016-11-24 11:49 UTC, movciari
no flags Details
rhevm.log (34.59 KB, text/plain)
2016-11-24 11:49 UTC, movciari
no flags Details
evm.log (303.35 KB, text/plain)
2016-11-24 11:50 UTC, movciari
no flags Details
production.log (26.33 KB, text/plain)
2016-11-24 11:51 UTC, movciari
no flags Details
automation.log (78.57 KB, text/plain)
2016-11-24 11:52 UTC, movciari
no flags Details

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?


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