Bug 1810091

Summary: [v2v][UI] It is possible to select RHV VM that is not running as conversion host
Product: Red Hat CloudForms Management Engine Reporter: Maayan Hadasi <mguetta>
Component: V2VAssignee: Mike Turley <mturley>
Status: CLOSED ERRATA QA Contact: Md Nadeem <mnadeem>
Severity: medium Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: low    
Version: 5.11.4CC: bthurber, dagur, fdupont, istein, mnadeem, simaishi
Target Milestone: GAKeywords: ZStream
Target Release: 5.11.5Flags: pm-rhel: cfme-5.11.z+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.11.5.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-05-05 13:43:32 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: V2V Target Upstream Version:
Embargoed:

Description Maayan Hadasi 2020-03-04 14:32:07 UTC
Description of problem:
UCI VMs that are not running listed under 'VMs to configure as conversion hosts' list


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


How reproducible:
100%


Steps to Reproduce:
1. Create an UCI VM in RHV
2. In UI, go to Migration Settings -> Conversion Hosts 
3. Configure new conversion host 


Actual results:
VMs that are not running are listed under 'VMs to configure as conversion hosts' list and can be selected

Comment 2 Fabien Dupont 2020-03-04 14:47:16 UTC
And if you try to configure it, the playbook will fail and the playbook log will show that the VM is unreachable.
It's a nice to have, so putting it as low priority.

Comment 3 Fabien Dupont 2020-03-25 09:38:51 UTC
Moving to 5.11.z as 5.11.6 target release is not created yet.

Comment 5 Mike Turley 2020-04-09 16:22:40 UTC
Fabien, was I correct in assuming we want to hide all powered-off VMs from this wizard (not just RHV UCI VMs, but all RHV and OSP VMs with power_state === 'off')?

Comment 6 Fabien Dupont 2020-04-09 17:12:54 UTC
Correct. We have no way to know that a VM is a UCI, so we need to hide all powered off VMs.

Comment 7 CFME Bot 2020-04-09 18:41:27 UTC
New commit detected on ManageIQ/manageiq-v2v/jansa:

https://github.com/ManageIQ/manageiq-v2v/commit/1d89fc5959c81596cecebe2e89b2757dc5209922
commit 1d89fc5959c81596cecebe2e89b2757dc5209922
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Wed Apr  8 11:57:47 2020 +0000
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Wed Apr  8 11:57:47 2020 +0000

    Merge pull request #1125 from mturley/bz1810091-hide-powered-off-vms

    [BZ#1810091] Conversion Host Wizard: hide powered-off VMs

    (cherry picked from commit 7092af67dbee92d056d6a57b3e846848b5bffbaa)

    https://bugzilla.redhat.com/show_bug.cgi?id=1810091

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardHostsStep/ConversionHostWizardHostsStep.js | 2 +
 1 file changed, 2 insertions(+)

Comment 11 errata-xmlrpc 2020-05-05 13:43:32 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:2020