Bug 1544764

Summary: Evacuate Host failed
Product: Red Hat CloudForms Management Engine Reporter: Ido Ovadia <iovadia>
Component: ProvidersAssignee: Scott Seago <sseago>
Status: CLOSED CURRENTRELEASE QA Contact: Ido Ovadia <iovadia>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: cpelland, gblomqui, jfrey, jhajyahy, jhardy, jprause, obarenbo, simaishi, slucidi, sseago
Target Milestone: GAKeywords: Regression, TestOnly, ZStream
Target Release: 5.10.0   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1553304 (view as bug list) Environment:
Last Closed: 2019-02-11 14:02:09 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Openstack Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1553304    
Attachments:
Description Flags
evacuate sceenshot
none
partial evm log
none
partial production log none

Description Ido Ovadia 2018-02-13 13:18:11 UTC
Created attachment 1395343 [details]
evacuate sceenshot

Description of problem:
======================
Evacuate Host failed

Evacuating Instance instance_test001 failed: Invalid input for field/attribute evacuate. Value: {u'host': u'', u'adminPass': u''}. 'onSharedStorage' is a required property

Version-Release number of selected component:
=============================================
CFME 5.9.0.20 RHOS10

How reproducible:
=================
100%

Steps to Reproduce:
===================
1. Snetup with 2*compute nodes, shared storage(external Ceph)
2. Shut down one compute node with running instance   
3. Browse: Compute --> clouds --> Instances 
4. Select an instance/s from the shut down compute node
5. Click Lifecycle --> Evacuate selected Instances

Actual results:
===============
Evacuating Instance instance_test001 failed: Invalid input for field/attribute evacuate. Value: {u'host': u'', u'adminPass': u''}. 'onSharedStorage' is a required property

Expected results:
=================
Instance evacuated successfully 

Additional info:
================
Screenshot and log are enclosed

Comment 1 Ido Ovadia 2018-02-13 13:19:23 UTC
Created attachment 1395345 [details]
partial evm log

Comment 2 Ido Ovadia 2018-02-13 13:20:01 UTC
Created attachment 1395347 [details]
partial production log

Comment 4 CFME Bot 2018-02-23 11:08:46 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/8ac6c1d7c81b33eefbc12a2b9a6ea1dccc097061

commit 8ac6c1d7c81b33eefbc12a2b9a6ea1dccc097061
Author:     Scott Seago <sseago>
AuthorDate: Wed Feb 21 12:59:54 2018 -0500
Commit:     Scott Seago <sseago>
CommitDate: Thu Feb 22 09:39:02 2018 -0500

    BZ1544764: Fix evacuate host param handling in controller
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1544764
    
    Fix mismatch in param value expectations for handling 'evacuate host'
    controller action.
    
    A related fix will be needed in fog-openstack to fix the on_shared_storage==false
    use case, but this should handle the 'true' use case.

 app/controllers/mixins/actions/vm_actions/evacuate.rb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comment 7 Jad Haj Yahya 2018-07-05 08:04:07 UTC
Verified on 5.10.0.2