Bug 1395753

Summary: [RFE] Improve import dialog for KVM based imports
Product: [oVirt] ovirt-engine Reporter: Tomáš Golembiovský <tgolembi>
Component: RFEsAssignee: Rob Young <royoung>
Status: CLOSED DEFERRED QA Contact: Lukas Svaty <lsvaty>
Severity: medium Docs Contact:
Priority: unspecified    
Version: futureCC: bugs, michal.skrivanek, rhodain
Target Milestone: ---Keywords: FutureFeature
Target Release: ---Flags: michal.skrivanek: ovirt-future?
rule-engine: planning_ack?
rule-engine: devel_ack?
rule-engine: testing_ack?
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: 2020-04-01 14:44:36 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 Tomáš Golembiovský 2016-11-16 15:21:58 UTC
The import dialog for KVM/libvirt based import is rather confusing. The problem is that you can specify username (and password) either as part of the URI or by filling the provided text fields. But these two methods are not interchangeable. Correct method depends on the transport protocol in use:

For SSH based imports you have to uncheck the 'Requires authentication' checkbox. You however still have to provide the user name (most likely 'root') as part of the URI, otherwise the connection will happen with user 'vdsm'. The existence of user name in URI is not checked and the behaviour is rather unexpected from user perspective. On the other hand specifying username/password in the provided text fields does not work; the username value is ignored and we don't support password-based logins.

For TCP/TLS it is the opposite. The user name provided in URI is ignored and username/password has to be provided via the text fields.

IMHO it would be better if the URI was constructed by  engine and import dialog would provide several fields to get the necessary info:

1) transport: ssh/tcp/tls
2) username
3) password
4) single text field for the path and extra parameters

Comment 1 Tomas Jelinek 2017-05-17 07:36:22 UTC
*** Bug 1451291 has been marked as a duplicate of this bug. ***

Comment 2 Michal Skrivanek 2020-03-18 15:43:45 UTC
This bug didn't get any attention for a while, we didn't have the capacity to make any progress. If you deeply care about it or want to work on it please assign/target accordingly

Comment 3 Michal Skrivanek 2020-03-18 15:46:59 UTC
This bug didn't get any attention for a while, we didn't have the capacity to make any progress. If you deeply care about it or want to work on it please assign/target accordingly

Comment 4 Michal Skrivanek 2020-04-01 14:44:36 UTC
ok, closing. Please reopen if still relevant/you want to work on it.

Comment 5 Michal Skrivanek 2020-04-01 14:49:34 UTC
ok, closing. Please reopen if still relevant/you want to work on it.