Change in vdsm[master]: Fix some pep8 errors in storageServer.py

smizrahi at redhat.com smizrahi at redhat.com
Thu Oct 18 14:37:14 UTC 2012


Saggi Mizrahi has posted comments on this change.

Change subject: Fix some pep8 errors in storageServer.py
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/storage/storageServer.py
Line 119: def _addIntegerOption(options, key, value):
Line 120:     try:
Line 121:         options.append("%s=%d" % (key, int(value)))
Line 122:     except ValueError:
Line 123:         raise se.InvalidParameterException(key, value)
Again...
Line 124: 
Line 125: class ExampleConnection(object):
Line 126:     """Do not inherit from this object it is just to show and document the
Line 127:     connection object interface"""


--
To view, visit http://gerrit.ovirt.org/8606
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9e40dbf8f441d83f5314921af4acba9980ec3f9f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list