Dan Kenigsberg has submitted this change and it was merged.

Change subject: configNetwork: fix bonding mode issue
......................................................................


configNetwork: fix bonding mode issue

Currently, we configure nics before bonding when create/edit bonding
device. That could cause the bonding mode can't be updated as specified.
Because, when we use ifup to activate the bonding slaves, it implicitly
creates the bonding device with the default mode 0, and adds the salve
nics to it. So you will see the following kernel message in system log:

bonding: unable to update mode of bondX because it has slaves

The fix is just to make bonding configured before nics. Then it can
update its mode because no slave attatches to it at that time.

Change-Id: I9855ef2d0e05603f07397eb26822fac5fc8eaf5b
Signed-off-by: Mark Wu <[email protected]>
---
M vdsm/configNetwork.py
1 file changed, 10 insertions(+), 9 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved
  Igor Lvovsky: Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9855ef2d0e05603f07397eb26822fac5fc8eaf5b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to