Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.
For bugs related to Red Hat Enterprise Linux 5 product line. The current stable release is 5.10. For Red Hat Enterprise Linux 6 and above, please visit Red Hat JIRA https://issues.redhat.com/secure/CreateIssue!default.jspa?pid=12332745 to report new issues.

Bug 372001

Summary: ambigous error message
Product: Red Hat Enterprise Linux 5 Reporter: Sage Grigull <mgrigull>
Component: python-virtinstAssignee: Cole Robinson <crobinso>
Status: CLOSED ERRATA QA Contact: Virtualization Bugs <virt-bugs>
Severity: low Docs Contact:
Priority: low    
Version: 5.1CC: ahecox, xen-maint
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: RHEA-2008-0371 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-05-21 16:45:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
5.2 virtinst validation overhaul which fixes this issue none

Description Sage Grigull 2007-11-09 01:23:15 UTC
Description of problem:
ambigous error message when creating a new virtual machine

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


How reproducible:
always

Steps to Reproduce:
1. select a "non-valid" source location e.g.
nfs:engineering.brisbane.redhat.com:/mnt/redhat/fedora/linux/releases/8/Fedora/i386/iso/Fedora-8-i386-DVD.iso
2. select a valid location for the 'storage space'
3.
  
Actual results:
Unable to complete install 'exceptions.RuntimeError Invalid install location
Traceback (most recent call last):
  File "/usr/share/virt-manager/virtManager/create.py", line 681, in do_install
    dom = guest.start_install(False, meter = meter)
  File "/usr/lib/python2.4/site-packages/virtinst/Guest.py", line 647, in
start_install
    tmpfiles = self._prepare_install_location(meter)
  File "/usr/lib/python2.4/site-packages/virtinst/ParaVirtGuest.py", line 65, in
_prepare_install_location
    (kernelfn,initrdfn,args) = DistroManager.acquireKernel(self.location, meter,
scratchdir=self.scratchdir, type=self.type)
  File "/usr/lib/python2.4/site-packages/virtinst/DistroManager.py", line 566,
in acquireKernel
    raise RuntimeError, "Invalid install location"
RuntimeError: Invalid install location
'

Expected results:
"invalid source media specification" or something allong those lines to
distinguish it from 'storage space" specification

Additional info:
found it was the source media spefication by creating a virtual machine which
did have a valid source.

Comment 1 Sage Grigull 2007-11-09 01:41:32 UTC
compare with:
install source set to:
nfs:engineering.brisbane.redhat.com:/mnt/redhat/fedora/linux/releases/8/Everything/

yeilding error message of:
Unable to complete install 'exceptions.RuntimeError Could not find an
installable distribution the install location
Traceback (most recent call last):
  File "/usr/share/virt-manager/virtManager/create.py", line 681, in do_install
    dom = guest.start_install(False, meter = meter)
  File "/usr/lib/python2.4/site-packages/virtinst/Guest.py", line 647, in
start_install
    tmpfiles = self._prepare_install_location(meter)
  File "/usr/lib/python2.4/site-packages/virtinst/ParaVirtGuest.py", line 65, in
_prepare_install_location
    (kernelfn,initrdfn,args) = DistroManager.acquireKernel(self.location, meter,
scratchdir=self.scratchdir, type=self.type)
  File "/usr/lib/python2.4/site-packages/virtinst/DistroManager.py", line 570,
in acquireKernel
    progresscb=progresscb, distro=distro, scratchdir=scratchdir)
  File "/usr/lib/python2.4/site-packages/virtinst/DistroManager.py", line 559,
in _storeForDistro
    raise RuntimeError, "Could not find an installable distribution the install
location"
RuntimeError: Could not find an installable distribution the install location
'

Comment 2 Cole Robinson 2007-12-18 21:58:11 UTC
I agree that the message is ambiguous, and actually I think the second message
needs some work too. How about in the first case (when virt-manager/virt-install
cannot even access the passed URI)

"Could not access the install path '%s'" where %s is the actual path entered.

And for the second case, where the passed path is an actual location but a valid
distro could not be found there:

"Could not find an installable distribution at '%s'" where %s is again the path
entered.

I think that should clear up the issues. Agreed?

Comment 4 RHEL Program Management 2007-12-20 16:04:58 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release.  Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products.  This request is not yet committed for inclusion in an Update
release.

Comment 5 Cole Robinson 2008-01-03 22:29:39 UTC
Created attachment 290797 [details]
5.2 virtinst validation overhaul which fixes this issue

Comment 6 Bill Burns 2008-01-03 23:53:33 UTC
Setting dec ack for Dan.


Comment 7 Cole Robinson 2008-01-07 20:43:41 UTC
Committed and built.

* Fri Jan 04 2008 Cole Robinson <crobinso> - 0.103.0-6.el5
- Validation and error message fixes across the board.
- (rhbz #201948 #249424 #372001 #221542)

Build                                     Tag                   Built by
----------------------------------------  --------------------  ----------------
python-virtinst-0.103.0-6.el5             dist-5E-qu-candidate  crobinso

Comment 10 errata-xmlrpc 2008-05-21 16:45:15 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHEA-2008-0371.html