answerfile: LACP bonding support

Signed-off-by: Yann Dirson <[email protected]>

answerfile.py

@@ -352,10 +352,35 @@ def parseFCoEInterface(self):
 
         return results
 
+    def parseBonding(self, nethw):
+        nodes = getElementsByTagName(self.top_node, ['bonding'])
+        if len(nodes) > 1:
+            raise AnswerfileException("<bonding> must appear only once")
+        if nodes:
+            node = nodes[0]
+            bond_name = getStrAttribute(node, ['name'], mandatory=True)
+            if bond_name != "bond0":
+                raise AnswerfileException("<bonding> name must be 'bond0'")
+
+            bond_mode = getStrAttribute(node, ['mode'], mandatory=True)
+            if bond_mode != "lacp":
+                raise AnswerfileException("<bonding> mode must be 'lacp'")
+
+            bond_members_str = getStrAttribute(node, ['members'], mandatory=True)
+            bond_members = bond_members_str.split(",")
+            if len(bond_members) < 2:
+                raise AnswerfileException("<bonding> members must be at least two")
+            for member in bond_members:
+                if member not in nethw:
+                    raise AnswerfileException("<bonding> member %r not in detected NICs" % (member,))
+
+            netutil.configure_bonding_interface(nethw, bond_name, bond_mode, bond_members)
+
     def parseInterface(self):
         results = {}
         node = getElementsByTagName(self.top_node, ['admin-interface'], mandatory=True)[0]
         nethw = netutil.scanConfiguration()
+        self.parseBonding(nethw)
 
         if_name = getStrAttribute(node, ['name'])
         if_hwaddr = getStrAttribute(node, ['hwaddr'])

doc/answerfile.txt

@@ -198,6 +198,14 @@ Format of 'source' and 'driver-source'
     Specifies additional devices to be included in the local SR.
 
 
+  <bonding name="bond0" mode="lacp" members="eth,eth[,eth]*"/>
+
+    Join given ethernet interfaces using LACP bonding.  Name of
+    resulting interface must be "bond0".  It can be then be used with
+
+      <admin-interface name="bond0"/>
+
+
   <admin-interface name="eth"/> | <admin-interface hwaddr="mac"/>
 
     Specifies the initial management interface.