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

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

 



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

_______________________________________________
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