Change in vdsm[master]: pep8: clean few small files

zhshzhou at linux.vnet.ibm.com zhshzhou at linux.vnet.ibm.com
Thu Oct 25 06:02:05 UTC 2012


Zhou Zheng Sheng has posted comments on this change.

Change subject: pep8: clean few small files
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/ksm.py
Line 72: 
Line 73:     def adjust(self):
Line 74:         """Adjust ksm's vigor
Line 75: 
Line 76:         according to configuration and current memory stress.
This docstring change is not related to PEP 8. I find the new docstring is more readable, so I think it's OK.
Line 77:         Return whether ksm is running"""
Line 78: 
Line 79:         self._lock.acquire()
Line 80:         try:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5326e6fb6a22979ab7d6cd6d4e61af746e9a6dd8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list