Bug 1422961

Summary: VM Migrate State Machine does not correctly report migration errors.
Product: Red Hat CloudForms Management Engine Reporter: Leo Khomenko <lkhomenk>
Component: AutomateAssignee: William Fitzgerald <wfitzger>
Status: CLOSED CURRENTRELEASE QA Contact: Ronny Pfannschmidt <ronny.pfannschmidt>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.7.0CC: hkataria, jhardy, lkhomenk, mkanoor, mpovolny, obarenbo, simaishi, tfitzger, wfitzger
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:migrate
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1479478 (view as bug list) Environment:
Last Closed: 2018-03-06 16:15:59 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1479478    

Description Leo Khomenko 2017-02-16 16:35:08 UTC
Description of problem:VM Migrate fails on RHEVM/VMware if no host was selected. but it's possible to create such Request


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


How reproducible:100%


Steps to Reproduce:
1.Migrate Vm without selecting Host on RHEVM/VMware

Actual results: Request gets created but fails


Expected results: Warning/Error message should be shown


Additional info:

Comment 3 William Fitzgerald 2017-05-24 18:29:57 UTC
I am working on this now.  We need to add enhanced messaging, error notification and error logging.

Comment 6 CFME Bot 2017-06-07 19:26:32 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/a4d4652cf8d2f1bf4af14ceae87f36f89318eb55

commit a4d4652cf8d2f1bf4af14ceae87f36f89318eb55
Author:     Tina Fitzgerald <tfitzger>
AuthorDate: Wed Jun 7 12:28:48 2017 -0400
Commit:     Tina Fitzgerald <tfitzger>
CommitDate: Wed Jun 7 12:28:48 2017 -0400

    Add vm_migrate_task factory.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1422961

 spec/factories/miq_request_task.rb | 3 +++
 1 file changed, 3 insertions(+)

Comment 7 CFME Bot 2017-06-08 19:54:09 UTC
New commit detected on ManageIQ/manageiq-automation_engine/master:
https://github.com/ManageIQ/manageiq-automation_engine/commit/aaf998e6b2b40503725fe35e9df75358f5eb99b2

commit aaf998e6b2b40503725fe35e9df75358f5eb99b2
Author:     Tina Fitzgerald <tfitzger>
AuthorDate: Wed Jun 7 12:47:15 2017 -0400
Commit:     Tina Fitzgerald <tfitzger>
CommitDate: Wed Jun 7 12:47:15 2017 -0400

    Check status to determine return value.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1422961

 .../miq_ae_service_vm_migrate_task.rb              |  2 +-
 ...roviders-vmware-infra_manager-provision_spec.rb | 50 ++++++++++++++++++++++
 2 files changed, 51 insertions(+), 1 deletion(-)

Comment 8 William Fitzgerald 2017-06-13 18:24:12 UTC
Still working on this.  Still missing a couple of things.

Thanks

Billy

Comment 9 Tina Fitzgerald 2017-08-08 15:35:46 UTC
This ticket initially reports an issue with required fields for a successful migration. We found issues in the VM migration state machine that needed to be resolved prior to addressing the reported issue of required fields. Splitting this ticket into 2 separate tickets to track both issues.

This ticket addresses the VM migrate state machine issues that prevented a failed migration from properly reporting the error.