Mark Wu has uploaded a new change for review.

Change subject: netconf: Sort bond's slave when objectivizing
......................................................................

netconf: Sort bond's slave when objectivizing

To make iproute2 configurator pass testBondHwAddress, we need keep
the bond's slaves sorted. Otherwise bond's hwaddr will vary with the
order of enslaving. This work is done by initscript for ifcfg
configurator.

Change-Id: I5db20ca598414ff29a368d99156b04b7f61961ae
Signed-off-by: Mark Wu <wu...@linux.vnet.ibm.com>
---
M vdsm/netmodels.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/91/19591/1

diff --git a/vdsm/netmodels.py b/vdsm/netmodels.py
index 50a5ba3..8cb2d3d 100644
--- a/vdsm/netmodels.py
+++ b/vdsm/netmodels.py
@@ -234,8 +234,8 @@
     def objectivize(cls, name, configurator, options, nics, mtu, _netinfo,
                     destroyOnMasterRemoval=None):
         if name and nics:  # New bonding or edit bonding.
-            slaves = cls._objectivizeSlaves(name, configurator, nics, mtu,
-                                            _netinfo)
+            slaves = cls._objectivizeSlaves(name, configurator, sorted(nics),
+                                            mtu, _netinfo)
             if name in _netinfo.bondings:
                 if _netinfo.ifaceUsers(name):
                     mtu = max(mtu, netinfo.getMtu(name))


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5db20ca598414ff29a368d99156b04b7f61961ae
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <wu...@linux.vnet.ibm.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to