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