Bug 1472843

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: Ilanit Stein <istein>
Component: ProvidersAssignee: Tomas Jelinek <tjelinek>
Status: CLOSED CURRENTRELEASE QA Contact: Ilanit Stein <istein>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: gblomqui, jfrey, jhardy, michal.skrivanek, nsimsolo, obarenbo, simaishi, tjelinek
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: rhev:v2v
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1512955 (view as bug list) Environment:
Last Closed: 2018-03-06 15:07:02 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: RHEVM Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1512955    
Attachments:
Description Flags
import_warning_message.png none

Description Ilanit Stein 2017-07-19 13:56:35 UTC
Description of problem:
On CFME/v2v dialog, Install drivers field should:
1. Not appear for a RHEL source VM.
2. Appear or a Windows source VM. 
   If there are no drivers installed, the driver install should be enforced 
   (That is do not allow to run the v2v operation, unless drivers are chosen to 
    be installed)
  
Version-Release number of selected component (if applicable):
CFME-5.8.1+v2v bug fixes.

Actual results:
The problem that we see now without having the above implemented,
that a windows source VM that do not have drivers installed, will be imported,
but will not be able to run windows (the disk type on destination should be VIRTIO, so even if source disk type was IDE, the virtio-win drivers are required)

Expected results:
There should be install drivers enforcement, for a windows VM, if drivers are not already installed in the source VM. 
Need to add a warning to user, when trying to submit the v2v operation, in such a case.

Comment 2 Michal Skrivanek 2017-07-20 06:40:39 UTC
(In reply to Ilanit Stein from comment #0)
> Description of problem:
> On CFME/v2v dialog, Install drivers field should:
> 1. Not appear for a RHEL source VM.

The OS identification is done by virt-v2v, it's best to let it decide if it needs anything from the drivers iso or not. 

> 2. Appear or a Windows source VM. 
>    If there are no drivers installed, the driver install should be enforced 
>    (That is do not allow to run the v2v operation, unless drivers are chosen
> to 
>     be installed)

that would be different from RHV - drivers are highly recommended, but not mandatory.

> Version-Release number of selected component (if applicable):
> CFME-5.8.1+v2v bug fixes.
> 
> Actual results:
> The problem that we see now without having the above implemented,
> that a windows source VM that do not have drivers installed, will be
> imported,
> but will not be able to run windows (the disk type on destination should be
> VIRTIO, so even if source disk type was IDE, the virtio-win drivers are
> required)
> 
> Expected results:
> There should be install drivers enforcement, for a windows VM, if drivers
> are not already installed in the source VM. 
> Need to add a warning to user, when trying to submit the v2v operation, in
> such a case.

Yes, warning is a good idea. That can be OS specific, since it's just a recommendation

Comment 3 Ilanit Stein 2017-07-20 13:33:18 UTC
Based on Michal's inputs from comment #2, and 
this input I got from Michal S./mbetak, on a separate session: 
On CFME side, there is no info on the drivers installed on the source VM, 

The following can't be implemented: 

1. enforcement of install drivers, if the required drivers are missing.
2. list only the relevant drivers in the drivers list.

Therefore, this bug is narrowed into: 
Add a warning similar to the warning we have on RHV,
when running v2v without installing drivers.

See attached RHV warning screenshot "import_warning_message.png"

Comment 4 Ilanit Stein 2017-07-20 13:34:22 UTC
Created attachment 1301745 [details]
import_warning_message.png

Comment 5 Tomas Jelinek 2017-08-11 14:09:21 UTC
It will be solved by checking the "Install Windows Drivers" checkbox by default when the source OS is windows. This way without selecting a driver you will get a validation message on dialog submit.

Comment 6 Michal Skrivanek 2017-08-24 11:48:48 UTC
isn't it good enough now in 5.8.2?

Comment 7 Tomas Jelinek 2017-08-24 11:52:57 UTC
yes, Id say so

Comment 13 Ilanit Stein 2018-01-01 10:51:31 UTC
Verified on CFME-5.9.0.14:
Windows VM: "Install Windows Drivers" checkbox is checked by default.
RHEL VM: "Install Windows Drivers" checkbox is NOT checked by default.