Bug 1395753 - [RFE] Improve import dialog for KVM based imports
Summary: [RFE] Improve import dialog for KVM based imports
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: RFEs
Version: future
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: ---
Assignee: Rob Young
QA Contact: Lukas Svaty
URL:
Whiteboard:
: 1451291 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-11-16 15:21 UTC by Tomáš Golembiovský
Modified: 2020-04-01 14:49 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-04-01 14:44:36 UTC
oVirt Team: Virt
Embargoed:
michal.skrivanek: ovirt-future?
rule-engine: planning_ack?
rule-engine: devel_ack?
rule-engine: testing_ack?


Attachments (Terms of Use)

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.


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