Re: [PATCH] Make lack of 'numvcpus' a warning only

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



John Levon wrote:
> On Tue, Jul 01, 2008 at 08:28:01PM +0100, Daniel P. Berrange wrote:
> 
>>> diff --git a/virt-unpack b/virt-unpack
>>> --- a/virt-unpack
>>> +++ b/virt-unpack
>>> @@ -119,7 +119,7 @@ def vmx_to_image_xml(disks_list,record,o
>>>      if record.has_key("numvcpus"):
>>>          vcpus = record["numvcpus"]
>>>      else:  
>>> -        logging.error("numvcpus key not parsed from %s, defaulting to 1 virtual cpu" % file)
>>> +        logging.warning("numvcpus key not parsed from %s, defaulting to 1 virtual cpu" % file)
>>>          vcpus = "1"
>> IMHO, we shoudn't bother logging anything just silently default
>> it to 1.
> 
> Fine by me. What about 'annotation'? I couldn't find this documented on
> sanbarrow.com
> 
> regards
> john
> 

Since annotation seems to just represent a machine description which can't
be an essential piece, I just removed the logging for it, as well as for
the vcpu default:

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

Thanks,
Cole

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

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux