FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora/Linux Management Tools

 
 
LinkBack Thread Tools
 
Old 08-20-2008, 11:23 PM
Boris Altshul
 
Default looks like a bug in LiveCdInstaller.py

I was trying to use some of myÂ*virsh based scripts on Fedora 9 and ran into problems. After some debugging, it looks like that
there is an issue with passing the parameter "os_type"Â*to the base class from LiveCdInstaller constructor. Take a look.
Â*
In LiveCdInstaller.py
-----------------------

class LiveCDInstaller(Guest.Installer):
Â* Â* def __init__(self, type = "xen", location = None, os_type = None):
Â* Â* Â* Â* Guest.Installer.__init__(self, type, location, os_type = None)
--------------------------------------------
Should be

class LiveCDInstaller(Guest.Installer):
Â* Â* def __init__(self, type = "xen", location = None, os_type = None):
Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* Â* -------VÂ* Â*
Â* Â* Â* Â* Guest.Installer.__init__(self, type, location, os_type=os_type)

_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@redhat.com
https://www.redhat.com/mailman/listinfo/et-mgmt-tools
 
Old 08-21-2008, 01:26 PM
Cole Robinson
 
Default looks like a bug in LiveCdInstaller.py

Boris Altshul wrote:
> I was trying to use some of my virsh based scripts on Fedora 9 and ran into problems. After some debugging, it looks like that
> there is an issue with passing the parameter "os_type"Â to the base class from LiveCdInstaller constructor. Take a look.
> In LiveCdInstaller.py
> -----------------------
>
> class LiveCDInstaller(Guest.Installer):
> Â Â def __init__(self, type = "xen", location = None, os_type = None):
> Â Â Â Â Guest.Installer.__init__(self, type, location, os_type = None)
> --------------------------------------------
> Should be
>
> class LiveCDInstaller(Guest.Installer):
> Â Â def __init__(self, type = "xen", location = None, os_type = None):
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â -------VÂ Â
> Â Â Â Â Guest.Installer.__init__(self, type, location, os_type=os_type)
>
>

Yes, thanks for the report. This was actually fixed upstream
a couple months ago.

http://hg.et.redhat.com/virt/applications/virtinst--devel?cs=7ce9673ddb72

We should be pushing a new release fairly soon, fyi.

Thanks,
Cole

_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@redhat.com
https://www.redhat.com/mailman/listinfo/et-mgmt-tools
 

Thread Tools




All times are GMT. The time now is 08:20 AM.

VBulletin, Copyright ©2000 - 2013, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright ©2007 - 2008, www.linux-archive.org