[virt-tools-list] virt-install cpuset=auto error

[Jared]

On 07/24/2012 08:12 PM, Jared wrote:
> On 07/24/2012 12:13 PM, Cole Robinson wrote:
>> Hmm. Just to make sure it isn't an install issue:
>> 
>> git clone git://git.fedorahosted.org/git/python-virtinst.git
>> cd python-virtinst
>> python setup.py build
>> ./virt-install --debug <the rest of your command>
>> 
>> and reply with the output attached.
>
> looks like doing it that way actually worked:
> 
> [Tue, 24 Jul 2012 20:09:48 virt-install 11669] DEBUG (cli:838) Auto cpuset
> is: 0,2,4,6,8,10,12,14,16,18,20,22


I did a diff of the git version and my installed version and found what you
see below.  Looks like this setting was shuffled around a bit so it could be
properly initialized.  You sure this wasn't added after the 0.600.2 release?

Either way, I'm good now.  I can patch my system version easily enough, and
as this is already fixed in the code I won't need to worry about it for the
next version.

Thanks a bunch for helping me track this down.

# diff -u /usr/bin/virt-install-2.7 ~/python-virtinst/virt-install
--- /usr/bin/virt-install-2.7   2012-07-21 20:30:53.000000000 -0500
+++ ~/python-virtinst/virt-install 2012-07-24 19:58:41.293789541 -0500
@@ -1,4 +1,4 @@
-#!/usr/bin/python2.7 -tt
+#!/usr/bin/python -tt
 #
 # Script to set up a Xen guest and kick off an install
 #
@@ -478,7 +478,6 @@
     # Guest configuration
     cli.get_uuid(options.uuid, guest)
     cli.get_vcpus(guest, options.vcpus, options.check_cpu)
-    cli.get_cpuset(guest, options.cpuset)
     cli.parse_numatune(guest, options.numatune)
     cli.parse_cpu(guest, options.cpu)
     cli.parse_security(guest, options.security)
@@ -534,6 +533,9 @@
     # this after setting guest.installer at least
     check_option_collisions(options, guest)

+    # Needs to come after setting memory
+    cli.get_cpuset(guest, options.cpuset, guest.memory)
+
     # Warnings
     if options.pxe and not supports_pxe(guest):
         logging.warn(_("The guest's network configuration does not support "

-- 
Jared