[et-mgmt-tools] [PATCH] Make lack of 'numvcpus' a warning only

[Cole Robinson]

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