better UX to validate KVM host addresses

Bug #1877140 reported by Junien F
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Expired
Medium
Unassigned

Bug Description

Hi,

MAAS 2.7 stable snap.

When editing a KVM host configuration, if I use a domain name (ie qemu+ssh://<email address hidden>/system) instead of an IP, clicking on "Save changes" will start a spinner for "Trying to connect and discover KVM" for a while and only then display a semi-cryptic "failed to detect a valid IP address from 'my-machine.internal'" error message

I think that :
1. the usage of an IP should be checked in Javascript on the browser side, and immediately error when pressing "Save changes"
2. Using a localhost-y IP (so any IP starting by 127. (and fe80::/64 , and ::1/128) ) should either be prohibited, or should trigger a warning about potential confusion if you have multiple controllers.
3. The error message should be better worded (for example : error : 'my-machine.internal' is not a valid IP address)

Thanks

Revision history for this message
Lee Trager (ltrager) wrote :

For LXD Pods in 2.8 we made MAAS a bit more forgivable. Both the protocol and port are optional. Instead of fixing this in the Javascript I think we should do the same for libvirt.

Changed in maas:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Jerzy Husakowski (jhusakowski) wrote :

Is this issue still present in recent MAAS (3.1 or later) and LXD as the VM host?

Changed in maas:
importance: High → Medium
status: Triaged → Incomplete
Junien F (axino)
Changed in maas:
status: Incomplete → Triaged
Revision history for this message
Jerzy Husakowski (jhusakowski) wrote :

The question whether it's still reproducible on recent MAAS still stands. MAAS 2.7 is out of support.

Changed in maas:
status: Triaged → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for MAAS because there has been no activity for 60 days.]

Changed in maas:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.