Change in vdsm[master]: Change of ConfigWriter._createConfFile parameters
asegurap at redhat.com
asegurap at redhat.com
Tue Aug 20 14:10:27 UTC 2013
Antoni Segura Puimedon has posted comments on this change.
Change subject: Change of ConfigWriter._createConfFile parameters
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
-1 for visibility of the question.
....................................................
File vdsm/netconf/ifcfg.py
Line 539: if not iface.mtu:
Line 540: mtu = confParams.get('MTU', None)
Line 541: if mtu:
Line 542: mtu = int(mtu)
Line 543: return iface.ip
Shouldn't this be iface.ipConfig so that we just return the named tuple?
Line 544:
Line 545: def removeNic(self, nic):
Line 546: cf = netinfo.NET_CONF_PREF + nic
Line 547: self._backup(cf)
--
To view, visit http://gerrit.ovirt.org/18331
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iae5ec15048ed52170371dda1d0fa9b90d51838e3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Ĺ ebek <psebek at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list