[PATCH] remove extraneous quotes from BONDING_OPTS
by Laine Stump
From: Satoru SATOH <ssato(a)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(a)sigxcpu.org>
Ed Maste <emaste(a)freebsd.org>
Hendrik Schwartke <hendrik(a)os-t.de>
- Ting Liu <b28495(a)freescale.com>
\ No newline at end of file
+ Ting Liu <b28495(a)freescale.com>
+ Satoru SATOH <ssato(a)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
10 years, 9 months