Change in vdsm[master]: added mtu support for network interface
danken at redhat.com
danken at redhat.com
Sun Feb 5 14:14:44 UTC 2012
Dan Kenigsberg has posted comments on this change.
Change subject: added mtu support for network interface
......................................................................
Patch Set 7: I would prefer that you didn't submit this
(2 inline comments)
two minor comments.
....................................................
File vdsm/configNetwork.py
Line 369: ''' method checks to see if the current bridge has MTU
docstring text should look like
"""Update MTU for devices that are to remain after "bridge" is removed
method checks to see if the current bridge has MTU .... etcetc
Line 561: configWriter.addBonding(bonding, ifaceBridge, bondingOptions=bondingOptions, mtu=mtu)
you should not reduce bonding mtu, too!
--
To view, visit http://gerrit.ovirt.org/754
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1d8f38627dd1a59830e426484228c1796081ad1b
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
More information about the vdsm-patches
mailing list