Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1849733

Summary:	[v2v][ui] VM import RHV to CNV: RHV URL pass validation though it is missing 'https://'
Product:	OpenShift Container Platform	Reporter:	Ilanit Stein <istein>
Component:	Console Kubevirt Plugin	Assignee:	Filip Krepinsky <fkrepins>
Status:	CLOSED DUPLICATE	QA Contact:	Daniel Gur <dagur>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	unspecified	CC:	aos-bugs, cnv-qe-bugs, pkliczew, pvauter, yzamir
Target Milestone:	---
Target Release:	4.6.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1850562 (view as bug list)		Environment:
Last Closed:	2020-06-24 14:00:59 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1850562

Description Ilanit Stein 2020-06-22 16:48:27 UTC

Description of problem:
When RHV URL is missing the 'https://' prefix,
and provide all other RHV details correctly (ca cert, user, password)
We get this message:
 "Connection succeeded but could not read list of virtual machines from the RHV API instance"
That prevents from loading the VMs in this RHV.

This should not happen, and user should be notified that the RHV URL is not valid.

We can do the following to solve this:

1. Add help message with example of a RHV URL - It can save users confusion
2. We can fail RHV URL on validation error 

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

Comment 1 Filip Krepinsky 2020-06-23 14:25:17 UTC

we can add the validation to the UI, but what protocols/ports should be allowed then? 

just https://hostname or hostname:443? What about http, 8443 or any other ports?

Would it make sense to default to http or https on backend?

Comment 2 Piotr Kliczewski 2020-06-23 14:36:37 UTC

We need https for importing the disks. As far as I know we can use either default https port (443) or 8443. I haven't seen any other port supported.

Comment 3 Yaacov Zamir 2020-06-24 14:00:59 UTC

Moving to Console Kubevirt Plugin

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

FYI:
@Nelly @Piotr @Tomas @Ilanit @Filip

*** This bug has been marked as a duplicate of bug 1850562 ***

Comment 4 Piotr Kliczewski 2020-06-24 14:30:22 UTC

Thanks