Change in vdsm[master]: configNet: support PREFIX option

asegurap at redhat.com asegurap at redhat.com
Wed Nov 21 09:50:17 UTC 2012


Antoni Segura Puimedon has posted comments on this change.

Change subject: configNet: support PREFIX option
......................................................................


Patch Set 3: Looks good to me, but someone else must approve

(1 inline comment)

I had not gone deep enough in my deployUtil.py.in reading to see that this version is good. Now I can approve.

....................................................
File vdsm/configNetwork.py
Line 892: 
Line 893:     if mtu:
Line 894:         mtu = int(mtu)
Line 895: 
Line 896:     prefix = int(options.get('prefix'))
AFAICT, as it is now, deployUtil.py.in will read the parameters in caps and feed them back in caps to addNetwork. However, it is true that _getBridgeParams will not encounter 'PREFIX' as it is not part of ifcfg-* file format.

For this reason. IMO The change is correct.
Line 897:     if prefix:
Line 898:         if netmask is None:
Line 899:             netmask = _prefix2netmask(prefix)
Line 900:             del options['prefix']


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1d87818642aa2092533f4864728c5ecd5d64f740
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list