I thins this is the correct formatting:
----
self.__putMetadata({"NONE": "#" * (sd.METASIZE - 10)}, metaid)
----
    cls.log.warn("Could not get size for vol %s/%s"
                 "using optimized methods",
                 sdobj.sdUUID, volUUID, exc_info=True)

----- Original Message -----
> From: "Deepak C Shetty" <deepa...@linux.vnet.ibm.com>
> To: "VDSM Project Development" <vdsm-devel@lists.fedorahosted.org>
> Sent: Tuesday, June 5, 2012 2:19:04 PM
> Subject: [vdsm] pep8 questions
> 
> Hi,
>      I was looking at resolving pep8 issues in
> vdsm/storage/blockVolume.py. Haven't been able to resolve the below..
> Pointers appreciated.
> 
> vdsm/storage/blockVolume.py:99:55: E225 missing whitespace around
> operator
> vdsm/storage/blockVolume.py:148:28: E201 whitespace after '{'
> vdsm/storage/blockVolume.py:207:28: E701 multiple statements on one
> line
> (colon)
> 
> 
> line 99:  cls.log.warn("Could not get size for vol %s/%s using
> optimized
> googling i found some links indicating this pep8 warning is
> incorrect.
> 
> line 148: cls.__putMetadata({ "NONE": "#" * (sd.METASIZE-10) },
> metaid)
> It gives some other error if i remove the whitespace after {
> 
> line 206 & 207:
>              raise se.VolumeCannotGetParent("blockVolume can't get
> parent %s for
>                    volume %s: %s" % (srcVolUUID, volUUID, str(e)))
> I split this line to overcome the > 80 error, but unable to decipher
> what this error means ?
> 
> thanx,
> deepak
> 
> _______________________________________________
> vdsm-devel mailing list
> vdsm-devel@lists.fedorahosted.org
> https://fedorahosted.org/mailman/listinfo/vdsm-devel
> 
_______________________________________________
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/vdsm-devel

Reply via email to