Dan Kenigsberg has uploaded a new change for review.

Change subject: tool: _enable_bond_dev: reopen bonding_masters per bond
......................................................................

tool: _enable_bond_dev: reopen bonding_masters per bond

Writing multiple +bondnames into /sys/class/net/bonding_masters is not
enough to add new bonding devices. One has to reopen that file for each
added bond.

Change-Id: I4693a3d2cc3beba0b5961d16fb1ef25170f99f9e
Signed-off-by: Dan Kenigsberg <[email protected]>
---
M vdsm-tool/load_needed_modules.py.in
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/09/12409/1

diff --git a/vdsm-tool/load_needed_modules.py.in 
b/vdsm-tool/load_needed_modules.py.in
index c24bc94..400ff1f 100644
--- a/vdsm-tool/load_needed_modules.py.in
+++ b/vdsm-tool/load_needed_modules.py.in
@@ -45,8 +45,8 @@
 
     # @ENGINENAME@ currently assumes that all bonding devices pre-exist
     existing = set(file(MASTER_FILE).read().split())
-    with open(MASTER_FILE, 'w') as f:
-        for bond in REQUIRED - existing:
+    for bond in REQUIRED - existing:
+        with open(MASTER_FILE, 'w') as f:
             f.write('+%s\n' % bond)
 
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4693a3d2cc3beba0b5961d16fb1ef25170f99f9e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to