Change in vdsm[master]: Remove redundant ifupping of bonded ifaces.

danken at redhat.com danken at redhat.com
Wed Dec 12 20:42:16 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: Remove redundant ifupping of bonded ifaces.
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(1 inline comment)

could you ask Meni to verify that this (after dropping nicSort) has no regressions? it sounds too good to be true..

....................................................
File vdsm/configNetwork.py
Line 1003: 
Line 1004:     # Now we can run ifup for all interfaces
Line 1005:     if bonding:
Line 1006:         ifup(bonding, bondBootproto == 'dhcp' and not blockingDhcp)
Line 1007:     else:
if not bonding:
  there is only one nic, so there is no point in nicSort'ing.
Line 1008:         # NICs must be activated in the same order of boot time
Line 1009:         # to expose the correct MAC address.
Line 1010:         for nic in nicSort(nics):
Line 1011:             ifup(nic, options.get('bootproto') == 'dhcp' and not blockingDhcp)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I069ac2b60a2d591b5cc76484e0d7eee30b1429e2
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: 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: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list