Change in vdsm[master]: Change getMaxMtu to use the info provided by sysfs

asegurap at redhat.com asegurap at redhat.com
Tue Jan 29 15:58:08 UTC 2013


Antoni Segura Puimedon has posted comments on this change.

Change subject: Change getMaxMtu to use the info provided by sysfs
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/configNetwork.py
Line 905:         configWriter = ConfigWriter()
Line 906: 
Line 907:     prevmtu = None
Line 908:     if mtu:
Line 909:         prevmtu = netinfo.getMaxMtu(nics)
Igor, could you remind me why do we check the mtu from configuration files and not just from netinfo?
Line 910: 
Line 911:     nic = nics[0] if nics else None
Line 912:     iface = bonding or nic
Line 913:     if bridged:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I89d007eb81c2c64e642ed80af26f882f7632dc1c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap 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