ctdb-daemon: Mark NoIPHostOnAllDisabled tunable as obsolete

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

diff --git a/ctdb/common/tunable.c b/ctdb/common/tunable.c
index 0b6f0266c72edffd831d9bdad0939f021a4a631b..4c1714cee2df4fae8087f081d014dd920f1e6691 100644 (file)
--- a/ctdb/common/tunable.c
+++ b/ctdb/common/tunable.c
@@ -141,7 +141,7 @@ static struct {
                offsetof(struct ctdb_tunable_list, db_size_warn) },
        { "PullDBPreallocation", 10*1024*1024, false,
                offsetof(struct ctdb_tunable_list, pulldb_preallocation_size) },
-       { "NoIPHostOnAllDisabled", 1, false,
+       { "NoIPHostOnAllDisabled", 1, true,
                offsetof(struct ctdb_tunable_list, no_ip_host_on_all_disabled) },
        { "Samba3AvoidDeadlocks", 0, true,
                offsetof(struct ctdb_tunable_list, samba3_hack) },

diff --git a/ctdb/doc/ctdb-tunables.7.xml b/ctdb/doc/ctdb-tunables.7.xml
index 75a3b01c14dbbca984ac23653f3bf79654912d98..a925ca5ab1f7fb2e2d4fe76df7839aa89de02946 100644 (file)
--- a/ctdb/doc/ctdb-tunables.7.xml
+++ b/ctdb/doc/ctdb-tunables.7.xml
@@ -442,29 +442,15 @@ MonitorInterval=20
       </para>
     </refsect2>
 
-    <refsect2>
-      <title>NoIPHostOnAllDisabled</title>
-      <para>Default: 0</para>
-      <para>
-       If no nodes are HEALTHY then by default ctdb will happily host
-       public IPs on disabled (unhealthy or administratively disabled)
-       nodes.  This can cause problems, for example if the underlying
-       cluster filesystem is not mounted.  When set to 1 and a node
-       is disabled, any IPs hosted by this node will be released and
-       the node will not takeover any IPs until it is no longer disabled.
-      </para>
-    </refsect2>
-
     <refsect2>
       <title>NoIPTakeover</title>
       <para>Default: 0</para>
       <para>
        When set to 1, ctdb will not allow IP addresses to be failed
        over to other nodes.  Any IP addresses already hosted on
-       healthy nodes will remain.  Usually IP addresses hosted on
-       unhealthy nodes will also remain, if NoIPHostOnAllDisabled is
-       0.  However, if NoIPHostOnAllDisabled is 1 then IP addresses
-       will be released by unhealthy nodes and will become un-hosted.
+       healthy nodes will remain.  Any IP addresses hosted on
+       unhealthy nodes will be released by unhealthy nodes and will
+       become un-hosted.
       </para>
     </refsect2>
 

diff --git a/ctdb/doc/ctdb.1.xml b/ctdb/doc/ctdb.1.xml
index 075add25880ff5be55a1be3d6d2a8bb00df2b5a5..90e532cb6897b4c3081b1246a62a4dd1ae3fa28b 100644 (file)
--- a/ctdb/doc/ctdb.1.xml
+++ b/ctdb/doc/ctdb.1.xml
@@ -827,7 +827,6 @@ DBRecordCountWarn       = 100000
 DBRecordSizeWarn        = 10000000
 DBSizeWarn              = 100000000
 PullDBPreallocation     = 10485760
-NoIPHostOnAllDisabled   = 0
 TDBMutexEnabled         = 1
 LockProcessesPerDB      = 200
 RecBufferSizeLimit      = 1000000

diff --git a/ctdb/server/ctdb_takeover_helper.c b/ctdb/server/ctdb_takeover_helper.c
index 9461e7149f0c98f503b1c4a1c31aef6b2f32f418..20687024a4068730ce54f3b1289f4f52a7c2b89b 100644 (file)
--- a/ctdb/server/ctdb_takeover_helper.c
+++ b/ctdb/server/ctdb_takeover_helper.c
@@ -672,7 +672,7 @@ static bool ipreallocated_recv(struct tevent_req *req, int *perr)
  * - Get nodemap
  * - Initialise IP allocation state.  Pass:
  *   + algorithm to be used;
- *   + various tunables (NoIPTakeover, NoIPFailback, NoIPHostOnAllDisabled)
+ *   + various tunables (NoIPTakeover, NoIPFailback)
  *   + list of nodes to force rebalance (internal structure, currently
  *     no way to fetch, only used by LCP2 for nodes that have had new
  *     IP addresses added).

diff --git a/ctdb/tests/tool/ctdb.listvars.001.sh b/ctdb/tests/tool/ctdb.listvars.001.sh
index ca014dcf9c3b6a56878d25d32534c4eb4bed2494..e9fb686e191aa713a127bddccfc954b72d90a5c6 100755 (executable)
--- a/ctdb/tests/tool/ctdb.listvars.001.sh
+++ b/ctdb/tests/tool/ctdb.listvars.001.sh
@@ -58,7 +58,6 @@ DBRecordCountWarn          = 100000
 DBRecordSizeWarn           = 10000000
 DBSizeWarn                 = 100000000
 PullDBPreallocation        = 10485760
-NoIPHostOnAllDisabled      = 1
 TDBMutexEnabled            = 1
 LockProcessesPerDB         = 200
 RecBufferSizeLimit         = 1000000