Antoni Segura Puimedon has posted comments on this change.
Change subject: Remove redundant ifupping of bonded ifaces.
......................................................................
Patch Set 1: (1 inline comment)
....................................................
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:
Oh, I was considering the possibility of a bridge with two nics, for example,
but I guess we don't support that use case of using that host as a networking
bridge.
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 <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Meni Yakove <[email protected]>
Gerrit-Reviewer: ShaoHe Feng <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
