Bug 1361193

Summary:	v2v: import dialog - SSL Verification field for VMware should be renamed to "verify server's SSL certificate"
Product:	[oVirt] ovirt-engine	Reporter:	Sharon Gratch <sgratch>
Component:	BLL.Virt	Assignee:	Sharon Gratch <sgratch>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Nisim Simsolo <nsimsolo>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	4.0.2	CC:	bugs, gklein, mgoldboi, nsimsolo, sbonazzo, tjelinek
Target Milestone:	ovirt-4.0.4	Flags:	rule-engine: ovirt-4.0.z+ mgoldboi: planning_ack+ tjelinek: devel_ack+ rule-engine: testing_ack+
Target Release:	4.0.4
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-09-26 12:34:30 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Virt	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Sharon Gratch 2016-07-28 13:22:44 UTC

Description of problem:
Import vm from VMware provider uses the "SSL verification" check box to determine if to add the "no_verify="1" flag to the URI or not.

If this checkbox is not checked (default) then the "no_verify=1' flag is added automatically to the URI and no VMware server's SSL certificate is verified.

if checked then the flag is not added to the URI and a verification of the server's SSL certificate is done.


Therefore the current field should renamed to: "verify server's SSL certificate". Otherwise it seems no SSL verification is done at all if not checked and it's not correct.

Relevant to "import" dialog and to "External Provider" dialog. 


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

How reproducible:
100%

Steps to Reproduce:
1.Open the "import Virtual machine(s) (v2v) dialog and select VMware external source OR open new/edit external provider and select VMware type.
2.see the "SSL verification" checkbox

Expected results:
Should be renamed to "verify server's SSL certificate".

Additional info:

Comment 2 Tomas Jelinek 2016-07-29 08:41:20 UTC

Also, this checkbox should be by default checked

Comment 3 Nisim Simsolo 2016-09-06 07:23:40 UTC

Verified:
rhevm-4.0.4-0.1.el7ev