Bug 1486461

Summary:	VM Migrate gets an error sending completion email.
Product:	Red Hat CloudForms Management Engine	Reporter:	William Fitzgerald <wfitzger>
Component:	Automate	Assignee:	William Fitzgerald <wfitzger>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Alex Newman <anewman>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	unspecified	CC:	anewman, jhardy, mkanoor, obarenbo, simaishi, tfitzger, wfitzger
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.9.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	5.9.0.1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1496937 (view as bug list)		Environment:
Last Closed:	2018-03-06 15:12:01 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:	1496937

Description William Fitzgerald 2017-08-29 19:49:29 UTC

Description of problem:
When you try to migrate a VM, an error occurs in Vm Migrate complete email.

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


How reproducible:
always

Steps to Reproduce:
1. Get an VMWARE/RHV environment where you can try to migrate A VM.
2. Migrate a VM, select host and datastore.
3. Check the UI and the log and you will see an error in email method

Actual results:
Error in email method

Expected results:
The email to be sent and the process to finish successfully

Additional info:

Comment 2 CFME Bot 2017-08-29 20:43:33 UTC

https://github.com/ManageIQ/manageiq-content/pull/177

Comment 3 Dave Johnson 2017-08-29 20:45:21 UTC

Please assess the impact of this issue and update the severity accordingly.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition.

If it's something like a tracker bug where it doesn't matter, please set it to Low/Low.

Comment 4 CFME Bot 2017-09-20 14:43:39 UTC

New commit detected on ManageIQ/manageiq-content/master:
https://github.com/ManageIQ/manageiq-content/commit/655f095d474d7620f37a3b5a4c7025f6e40b9e0a

commit 655f095d474d7620f37a3b5a4c7025f6e40b9e0a
Author:     william fitzgerald <wfitzger>
AuthorDate: Tue Aug 29 16:00:38 2017 -0400
Commit:     william fitzgerald <wfitzger>
CommitDate: Wed Sep 20 10:24:00 2017 -0400

    Fix VM Migrate complete email when To field is nil.
    
    Made change to not send email when the To field is nil.
    Added log message that email will not be sent.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1486461

 .../Email.class/__methods__/vmmigratetask_complete.rb      |  7 ++++---
 .../method_validation/vm_migrate_task_complete_spec.rb     | 14 +++++++++++---
 2 files changed, 15 insertions(+), 6 deletions(-)