Bug 1849733 - [v2v][ui] VM import RHV to CNV: RHV URL pass validation though it is missing 'https://'
Summary: [v2v][ui] VM import RHV to CNV: RHV URL pass validation though it is missing ...
Keywords:
Status: CLOSED DUPLICATE of bug 1850562
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Console Kubevirt Plugin
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 4.6.0
Assignee: Filip Krepinsky
QA Contact: Daniel Gur
URL:
Whiteboard:
Depends On:
Blocks: 1850562
TreeView+ depends on / blocked
 
Reported: 2020-06-22 16:48 UTC by Ilanit Stein
Modified: 2020-09-08 13:48 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1850562 (view as bug list)
Environment:
Last Closed: 2020-06-24 14:00:59 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github ManageIQ manageiq-v2v-conversion_host pull 75 0 None closed Handle urls when protocol is not provided 2020-09-16 15:28:15 UTC

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


Note You need to log in before you can comment on or make changes to this bug.