Bug 460585 - update virtinst to install f10 xen guests
Summary: update virtinst to install f10 xen guests
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: python-virtinst
Version: 5.2
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: Cole Robinson
QA Contact: Virtualization Bugs
URL:
Whiteboard:
: 471079 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-08-28 19:42 UTC by Cole Robinson
Modified: 2009-12-14 21:10 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-01-20 21:04:51 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Parse treeinfo file if located in tree and add f10 os entry (6.85 KB, text/plain)
2008-09-15 19:42 UTC, Cole Robinson
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2009:0216 0 normal SHIPPED_LIVE python-virtinst enhancement and bug fix update 2009-01-20 16:06:21 UTC

Description Cole Robinson 2008-08-28 19:42:44 UTC
virtinst will need to be fixed to install fedora 10 xen guests. the only real solution is to backport the treeinfo parsing code and pull out the images-xen entry for PV guests.

Comment 1 RHEL Program Management 2008-08-28 19:52:12 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 2 Orion Poplawski 2008-09-04 22:44:23 UTC
I'm happy to test any rpms.  I'd really like to be able to install F10/rawhide on my EL5 machine.

Comment 4 Cole Robinson 2008-09-15 19:42:06 UTC
Created attachment 316777 [details]
Parse treeinfo file if located in tree and add f10 os entry

Comment 5 Orion Poplawski 2008-09-16 15:44:28 UTC
Hmm, doesn't apply cleanly to python-virtinst-0.300.2-8.el5:

patch -b -p1 < ~/virtinst-0.300.2-f10-treeinfo.patch 
patching file virtinst/FullVirtGuest.py              
Hunk #1 FAILED at 45.                                
1 out of 1 hunk FAILED -- saving rejects to file virtinst/FullVirtGuest.py.rej
patching file virtinst/OSDistro.py                                            
Hunk #2 succeeded at 51 (offset -1 lines).                                    
Hunk #3 succeeded at 109 with fuzz 2.                                         

FullVirtGuest.py fails because 0.300.2-8 doesn't list fedora9 as well.  Added that.

Then:

virt-install -n xenfdev64 -r 512 --vcpus=1 -m 40:00:00:00:00:03 --arch=x86_64 -p -f /dev/rootvg/xenfdev64 -l http://fedora.cora.nwra.com/fedora/linux/development/x86_64/os -x ks=http://sw.cora.nwra.com/ks/rawhide-nox64.cfg
Would you like to enable graphics support? (yes or no) yes


Starting install...
__init__() takes at most 4 arguments (5 given)
Domain installation may not have been
 successful.  If it was, you can restart your domain
 by running 'virsh start xenfdev64'; otherwise, please
 restart your installation.
Tue, 16 Sep 2008 09:26:45 ERROR    __init__() takes at most 4 arguments (5 given)
Traceback (most recent call last):
  File "/usr/sbin/virt-install", line 502, in ?
    main()
  File "/usr/sbin/virt-install", line 462, in main
    dom = guest.start_install(conscb,progresscb)
  File "/usr/lib/python2.4/site-packages/virtinst/Guest.py", line 811, in start_install
    self._prepare_install(meter)
  File "/usr/lib/python2.4/site-packages/virtinst/ParaVirtGuest.py", line 56, in _prepare_install
    self._installer.prepare(guest = self, meter = meter)
  File "/usr/lib/python2.4/site-packages/virtinst/DistroManager.py", line 239, in prepare
    self._prepare_kernel_and_initrd(guest, distro, meter)
  File "/usr/lib/python2.4/site-packages/virtinst/DistroManager.py", line 209, in _prepare_kernel_and_initrd
    distro = distro)
  File "/usr/lib/python2.4/site-packages/virtinst/DistroManager.py", line 115, in acquireKernel
    scratchdir=scratchdir)
  File "/usr/lib/python2.4/site-packages/virtinst/DistroManager.py", line 60, in _checkRHDistro
    store = FedoraDistro(baseuri, type, scratchdir)
  File "/usr/lib/python2.4/site-packages/virtinst/OSDistro.py", line 55, in __init__
    Distro.__init__(self, uri, type, scratchdir, arch)
TypeError: __init__() takes at most 4 arguments (5 given)

so, dropped arch from Distro.__init__:

        Distro.__init__(self, uri, type, scratchdir)



Now to get images-xen fixed in .treeinfo again...

Comment 6 Cole Robinson 2008-09-16 15:51:29 UTC
Sorry, should have mentioned that this was applied dependent on some other changes.

You will probably need to apply the patches in bz 448634 and bz 449129 in order for this to apply cleanly. :( Lot's of mucking around in this section of the code it seems.

Comment 7 Orion Poplawski 2008-09-16 15:53:09 UTC
Is there a final src.rpm somewhere I could get?

Comment 8 Cole Robinson 2008-09-18 21:01:12 UTC
This is built in python-virtinst-0.300.2-10.el5. Setting to MODIFIED.

Comment 9 Orion Poplawski 2008-09-18 21:02:58 UTC
Is there somewhere I can get a copy of python-virtinst-0.300.2-10.el5?

Comment 12 Daniel Berrangé 2008-11-11 18:52:10 UTC
*** Bug 471079 has been marked as a duplicate of this bug. ***

Comment 17 errata-xmlrpc 2009-01-20 21:04:51 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 therefore 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/RHBA-2009-0216.html


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