Change in vdsm[master]: Yet another bunch of completely pep8 fixed files.

danken at redhat.com danken at redhat.com
Sat Dec 15 12:14:43 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: Yet another bunch of completely pep8 fixed files.
......................................................................


Patch Set 2: Looks good to me, approved

(1 inline comment)

....................................................
File Makefile.am
Line 51: 	vds_bootstrap/miniyum.py \
Line 52: 	vdsm-tool \
Line 53: 	vdsm/*.py \
Line 54: 	vdsm/*.py.in \
Line 55: 	vdsm/sos/vdsm.py.in \
I do not understand the issue - we have several directories listed here anyway.

I suppose this patch can be taken even without my understanding.
Line 56: 	vdsm/storage/__init__.py \
Line 57: 	vdsm/storage/blockVolume.py \
Line 58: 	vdsm/storage/devicemapper.py \
Line 59: 	vdsm/storage/domainMonitor.py \


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2d0e38fd43bd76febf3bfb231c4257298e6d921f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list