[netcf-devel] [PATCH] remove extraneous quotes from BONDING_OPTS

Laine Stump laine at laine.org
Wed Jul 3 17:37:00 UTC 2013


From: Satoru SATOH <ssato at redhat.com>

For some unknown reason, the BONDING_OPTS setting in ifcfg files was
being set with an extra set of single quotes. So, for example, instead of:

  BONDING_OPTS="mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0"

we would get (e.g.):

  BONDING_OPTS="'mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0'"

Even when there weren't any BONDING_OPTS to set, the ifcfg file would
still get:

  BONDING_OPTS="''"

Since the extra quotes are added in all cases, and are never needed,
this patch just unconditionally removes them.
---
 AUTHORS                               | 3 ++-
 data/xml/util-get.xsl                 | 2 --
 tests/redhat/schema/bond-arp.xml      | 2 +-
 tests/redhat/schema/bond-defaults.xml | 2 +-
 tests/redhat/schema/bond.xml          | 2 +-
 tests/redhat/schema/bridge-bond.xml   | 2 +-
 tests/suse/schema/bond-defaults.xml   | 2 +-
 tests/suse/schema/bond.xml            | 2 +-
 tests/suse/schema/bridge-bond.xml     | 2 +-
 9 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/AUTHORS b/AUTHORS
index 08df6a0..d5a37a2 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -19,4 +19,5 @@ Contributions by:
   Guido Günther    <agx at sigxcpu.org>
   Ed Maste         <emaste at freebsd.org>
   Hendrik Schwartke <hendrik at os-t.de>
-  Ting Liu         <b28495 at freescale.com>
\ No newline at end of file
+  Ting Liu         <b28495 at freescale.com>
+  Satoru SATOH     <ssato at redhat.com>
diff --git a/data/xml/util-get.xsl b/data/xml/util-get.xsl
index 9edace3..ac0a320 100644
--- a/data/xml/util-get.xsl
+++ b/data/xml/util-get.xsl
@@ -3,7 +3,6 @@
                 version="1.0">
 
   <xsl:template name="bonding-opts">
-    <xsl:text>'</xsl:text>
     <xsl:if test="bond/@mode">mode=<xsl:value-of select='bond/@mode'/></xsl:if>
     <xsl:if test="bond/@mode = 'active-backup'"> primary=<xsl:value-of select='bond/interface[1]/@name'/></xsl:if>
     <xsl:if test="bond/miimon">
@@ -21,7 +20,6 @@
       <xsl:text> arp_ip_target=</xsl:text><xsl:value-of select="bond/arpmon/@target"/>
       <xsl:if test="bond/arpmon/@validate"><xsl:text> arp_validate=</xsl:text><xsl:value-of select="bond/arpmon/@validate"/></xsl:if>
     </xsl:if>
-    <xsl:text>'</xsl:text>
   </xsl:template>
 
 </xsl:stylesheet>
diff --git a/tests/redhat/schema/bond-arp.xml b/tests/redhat/schema/bond-arp.xml
index 6a53d09..884cb84 100644
--- a/tests/redhat/schema/bond-arp.xml
+++ b/tests/redhat/schema/bond-arp.xml
@@ -7,7 +7,7 @@
     <node label="IPADDR" value="192.168.50.7"/>
     <node label="NETMASK" value="255.255.255.0"/>
     <node label="GATEWAY" value="192.168.50.1"/>
-    <node label="BONDING_OPTS" value="'mode=active-backup primary=eth1 arp_interval=100 arp_ip_target=192.168.50.1 arp_validate=active'"/>
+    <node label="BONDING_OPTS" value="mode=active-backup primary=eth1 arp_interval=100 arp_ip_target=192.168.50.1 arp_validate=active"/>
   </tree>
   <tree path="/files/etc/sysconfig/network-scripts/ifcfg-eth1">
     <node label="DEVICE" value="eth1"/>
diff --git a/tests/redhat/schema/bond-defaults.xml b/tests/redhat/schema/bond-defaults.xml
index 3e92d19..5ee875d 100644
--- a/tests/redhat/schema/bond-defaults.xml
+++ b/tests/redhat/schema/bond-defaults.xml
@@ -11,7 +11,7 @@
     <node label="IPADDR" value="192.168.50.7"/>
     <node label="NETMASK" value="255.255.255.0"/>
     <node label="GATEWAY" value="192.168.50.1"/>
-    <node label="BONDING_OPTS" value="''"/>
+    <node label="BONDING_OPTS" value=""/>
   </tree>
   <tree  path="/files/etc/sysconfig/network-scripts/ifcfg-eth1">
     <node label="DEVICE" value="eth1"/>
diff --git a/tests/redhat/schema/bond.xml b/tests/redhat/schema/bond.xml
index d97542d..4a7e9bf 100644
--- a/tests/redhat/schema/bond.xml
+++ b/tests/redhat/schema/bond.xml
@@ -11,7 +11,7 @@
     <node label="IPADDR" value="192.168.50.7"/>
     <node label="NETMASK" value="255.255.255.0"/>
     <node label="GATEWAY" value="192.168.50.1"/>
-    <node label="BONDING_OPTS" value="'mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0'"/>
+    <node label="BONDING_OPTS" value="mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0"/>
   </tree>
   <tree  path="/files/etc/sysconfig/network-scripts/ifcfg-eth1">
     <node label="DEVICE" value="eth1"/>
diff --git a/tests/redhat/schema/bridge-bond.xml b/tests/redhat/schema/bridge-bond.xml
index c72f8d3..0b521d4 100644
--- a/tests/redhat/schema/bridge-bond.xml
+++ b/tests/redhat/schema/bridge-bond.xml
@@ -17,7 +17,7 @@
     <node label="DEVICE" value="bond0"/>
     <node label="ONBOOT" value="yes"/>
     <node label="MTU" value="1500"/>
-    <node label="BONDING_OPTS" value="'mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0'"/>
+    <node label="BONDING_OPTS" value="mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0"/>
     <node label="BRIDGE" value="br0"/>
   </tree>
   <tree path="/files/etc/sysconfig/network-scripts/ifcfg-eth1">
diff --git a/tests/suse/schema/bond-defaults.xml b/tests/suse/schema/bond-defaults.xml
index 3e92d19..5ee875d 100644
--- a/tests/suse/schema/bond-defaults.xml
+++ b/tests/suse/schema/bond-defaults.xml
@@ -11,7 +11,7 @@
     <node label="IPADDR" value="192.168.50.7"/>
     <node label="NETMASK" value="255.255.255.0"/>
     <node label="GATEWAY" value="192.168.50.1"/>
-    <node label="BONDING_OPTS" value="''"/>
+    <node label="BONDING_OPTS" value=""/>
   </tree>
   <tree  path="/files/etc/sysconfig/network-scripts/ifcfg-eth1">
     <node label="DEVICE" value="eth1"/>
diff --git a/tests/suse/schema/bond.xml b/tests/suse/schema/bond.xml
index d97542d..4a7e9bf 100644
--- a/tests/suse/schema/bond.xml
+++ b/tests/suse/schema/bond.xml
@@ -11,7 +11,7 @@
     <node label="IPADDR" value="192.168.50.7"/>
     <node label="NETMASK" value="255.255.255.0"/>
     <node label="GATEWAY" value="192.168.50.1"/>
-    <node label="BONDING_OPTS" value="'mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0'"/>
+    <node label="BONDING_OPTS" value="mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0"/>
   </tree>
   <tree  path="/files/etc/sysconfig/network-scripts/ifcfg-eth1">
     <node label="DEVICE" value="eth1"/>
diff --git a/tests/suse/schema/bridge-bond.xml b/tests/suse/schema/bridge-bond.xml
index c72f8d3..0b521d4 100644
--- a/tests/suse/schema/bridge-bond.xml
+++ b/tests/suse/schema/bridge-bond.xml
@@ -17,7 +17,7 @@
     <node label="DEVICE" value="bond0"/>
     <node label="ONBOOT" value="yes"/>
     <node label="MTU" value="1500"/>
-    <node label="BONDING_OPTS" value="'mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0'"/>
+    <node label="BONDING_OPTS" value="mode=active-backup primary=eth1 miimon=100 updelay=10 use_carrier=0"/>
     <node label="BRIDGE" value="br0"/>
   </tree>
   <tree path="/files/etc/sysconfig/network-scripts/ifcfg-eth1">
-- 
1.7.11.7



More information about the netcf-devel mailing list