Bug 1512955

Summary: [v2v] Add a warning to user, in case trying to run v2v for windows VM, without installing the required drivers
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: ProvidersAssignee: Tomas Jelinek <tjelinek>
Status: CLOSED ERRATA QA Contact: Ilanit Stein <istein>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: cpelland, gblomqui, jfrey, jhardy, michal.skrivanek, obarenbo, simaishi, tjelinek
Target Milestone: GAKeywords: ZStream
Target Release: 5.8.3   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: rhev:v2v
Fixed In Version: 5.8.3.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1472843 Environment:
Last Closed: 2018-02-28 13:08:47 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: RHEVM Target Upstream Version:
Embargoed:
Bug Depends On: 1472843    
Bug Blocks:    

Comment 2 CFME Bot 2017-11-14 14:01:26 UTC 
New commit detected on ManageIQ/manageiq/fine:
https://github.com/ManageIQ/manageiq/commit/2b879f88e39754d9e9a7878cd96fba36153114c4

commit 2b879f88e39754d9e9a7878cd96fba36153114c4
Author:     Greg McCullough <gmccullo>
AuthorDate: Tue Sep 5 15:51:07 2017 -0400
Commit:     Satoe Imaishi <simaishi>
CommitDate: Tue Nov 14 09:01:31 2017 -0500

    Merge pull request #76 from matobet/dynamic-checkbox
    
    v2v: Make "install drivers" checkbox dynamic
    (cherry picked from commit 0273a003fde7f1c52fa117ef1fe355303895a607)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1512955

 product/dialogs/service_dialogs/transform-vm.yml | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
Comment 3 CFME Bot 2017-11-14 14:03:54 UTC 
New commit detected on ManageIQ/manageiq-content/fine:
https://github.com/ManageIQ/manageiq-content/commit/2e49226616778e04f5a2524c3ff9807494436016

commit 2e49226616778e04f5a2524c3ff9807494436016
Author:     Greg McCullough <gmccullo>
AuthorDate: Thu Aug 31 17:04:19 2017 -0400
Commit:     Satoe Imaishi <simaishi>
CommitDate: Tue Nov 14 08:58:45 2017 -0500

    Merge pull request #170 from matobet/dynamic-windows
    
    v2v: Pre-check install_drivers checkbox for windows VMs
    (cherry picked from commit efd5d06b76ca7c581586a122ccb184c891e49b43)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1512955

 .../Import.class/__methods__/install_drivers.rb    | 34 +++++++++++++
 .../Import.class/__methods__/install_drivers.yaml  | 12 +++++
 .../VM/Transform/Import.class/install_drivers.yaml | 12 +++++
 .../__methods__/install_drivers_spec.rb            | 55 ++++++++++++++++++++++
 4 files changed, 113 insertions(+)
 create mode 100644 content/automate/ManageIQ/Infrastructure/VM/Transform/Import.class/__methods__/install_drivers.rb
 create mode 100644 content/automate/ManageIQ/Infrastructure/VM/Transform/Import.class/__methods__/install_drivers.yaml
 create mode 100644 content/automate/ManageIQ/Infrastructure/VM/Transform/Import.class/install_drivers.yaml
 create mode 100644 spec/content/automate/ManageIQ/Infrastructure/VM/Transform/Import.class/__methods__/install_drivers_spec.rb
Comment 4 Tomas Jelinek 2018-01-09 09:46:29 UTC 
both patches merged to the fine branch, moving to post
Comment 5 Ilanit Stein 2018-01-17 12:33:57 UTC 
Verified on CFME-5.8.3.1\RHV-4.1.8:
Windows VM: "Install Windows Drivers" checkbox is checked by default.
RHEL VM: "Install Windows Drivers" checkbox is NOT checked by default.
Comment 8 errata-xmlrpc 2018-02-28 13:08:47 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/RHSA-2018:0374