ctdb-scripts: Drop CTDB_PUBLIC_INTERFACE configuration option

The interface must always be specified in the public addresses file.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>

diff --git a/ctdb/config/ctdbd_wrapper b/ctdb/config/ctdbd_wrapper
index d65d53c78fac9a4c824f527e7404e30045a2ba65..48aad782bd9d1ecaa2f2c847c1cce8e0d2e6b7f9 100755 (executable)
--- a/ctdb/config/ctdbd_wrapper
+++ b/ctdb/config/ctdbd_wrapper
@@ -95,7 +95,6 @@ build_ctdb_options ()
     maybe_set "--logging"                "$CTDB_LOGGING"
     maybe_set "--nlist"                  "$CTDB_NODES"
     maybe_set "--listen"                 "$CTDB_NODE_ADDRESS"
-    maybe_set "--public-interface"       "$CTDB_PUBLIC_INTERFACE"
     maybe_set "--dbdir"                  "$CTDB_DBDIR"
     maybe_set "--dbdir-persistent"       "$CTDB_DBDIR_PERSISTENT"
     maybe_set "--dbdir-state"            "$CTDB_DBDIR_STATE"

diff --git a/ctdb/config/events.d/10.interface b/ctdb/config/events.d/10.interface
index 7093470e8ea3af8e8c0d20672f6a3c63131710be..f05559b22ae643415919c739d6f7cfffa9d8cf01 100755 (executable)
--- a/ctdb/config/events.d/10.interface
+++ b/ctdb/config/events.d/10.interface
@@ -29,9 +29,6 @@ get_all_interfaces ()
                         -e "s/,/ /g" \
                         -e "s/[\t ]*$//" "$ctdb_public_addresses")
 
-    # Add some special interfaces if they're defined
-    [ "$CTDB_PUBLIC_INTERFACE" ] && all_interfaces="$CTDB_PUBLIC_INTERFACE $all_interfaces"
-
     # Get the interfaces for which CTDB has public IPs configured.
     # That is, for all but the 1st line, get the 1st field.
     ctdb_ifaces=$($CTDB -X ifaces | sed -e '1d' -e 's@^|@@' -e 's@|.*@@')

diff --git a/ctdb/doc/ctdbd.conf.5.xml b/ctdb/doc/ctdbd.conf.5.xml
index 4fe01ef6de368789a6d231fc8bbdf3c16c759e13..652bb31bdea11fd97511e40ea5eb39252df36783 100644 (file)
--- a/ctdb/doc/ctdbd.conf.5.xml
+++ b/ctdb/doc/ctdbd.conf.5.xml
@@ -325,16 +325,6 @@
        </listitem>
       </varlistentry>
 
-      <varlistentry>
-       <term>CTDB_PUBLIC_INTERFACE=<parameter>INTERFACE</parameter></term>
-       <listitem>
-         <para>
-           No default.  Corresponds to
-           <option>--public-interface</option>.
-         </para>
-       </listitem>
-      </varlistentry>
-
       <varlistentry>
        <term>CTDB_RECOVERY_LOCK=<parameter>LOCK</parameter></term>
        <listitem>

diff --git a/ctdb/tests/eventscripts/10.interface.monitor.007.sh b/ctdb/tests/eventscripts/10.interface.monitor.007.sh
deleted file mode 100755 (executable)
index c45900e..0000000
--- a/ctdb/tests/eventscripts/10.interface.monitor.007.sh
+++ /dev/null
@@ -1,13 +0,0 @@
-#!/bin/sh
-
-. "${TEST_SCRIPTS_DIR}/unit.sh"
-
-define_test "unknown interface, up"
-
-setup_ctdb
-
-export CTDB_PUBLIC_INTERFACE="dev999"
-
-ok_null
-
-simple_test

diff --git a/ctdb/tests/eventscripts/10.interface.monitor.008.sh b/ctdb/tests/eventscripts/10.interface.monitor.008.sh
deleted file mode 100755 (executable)
index 8ab84f8..0000000
--- a/ctdb/tests/eventscripts/10.interface.monitor.008.sh
+++ /dev/null
@@ -1,21 +0,0 @@
-#!/bin/sh
-
-. "${TEST_SCRIPTS_DIR}/unit.sh"
-
-define_test "unknown interface, down, up"
-
-setup_ctdb
-
-iface="dev999"
-export CTDB_PUBLIC_INTERFACE="$iface"
-
-ethtool_interfaces_down "$iface"
-required_result 1 "ERROR: No link on the public network interface $iface"
-simple_test
-
-ethtool_interfaces_up "$iface"
-ok_null
-simple_test
-
-ok_null
-simple_test