Dan Kenigsberg has posted comments on this change.
Change subject: pep8: more semi-automatic fixes
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File vdsm/storage/iscsi.py
Line 246:
Line 247: # Technically there are a lot more interface properties but VDSM doesn't
Line 248:
Line 249:
Line 250: # support them at the moment
alas, I had an off-by-one bug in my script.
Line 251: class IscsiInterface(object):
Line 252:
Line 253: _fields = {
Line 254: "name": ('iface.iscsi_ifacename', 'rw'),
--
To view, visit
http://gerrit.ovirt.org/9606
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifbb36be20c77f714246279d1ae9067353bced047
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou(a)linux.vnet.ibm.com>