ctdb-tools: Drop ctdb --socket option
authorMartin Schwenke <martin@meltin.net>
Wed, 21 Feb 2018 10:33:49 +0000 (21:33 +1100)
committerAmitay Isaacs <amitay@samba.org>
Mon, 19 Mar 2018 01:23:18 +0000 (02:23 +0100)
Use environment variables for test-only options.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
ctdb/doc/ctdb.1.xml
ctdb/tools/ctdb.c

index 3103d62cc1f22c4019d008b8041ea75bb1e7d8d7..ae30cd10c7d75ac55fb56326c61f56dba3cd4942 100644 (file)
        </para>
       </listitem>
       </varlistentry>
-
-      <varlistentry><term>--socket=<parameter>FILENAME</parameter></term>
-      <listitem>
-       <para>
-         Specify that FILENAME is the name of the Unix domain
-         socket to use when connecting to the local CTDB
-         daemon. The default is
-         <filename>/usr/local/var/run/ctdb/ctdbd.socket</filename>.
-       </para>
-      </listitem>
-      </varlistentry>
     </variablelist>
   </refsect1>
 
index d76d17612102262b2ec8282888119942a31a3de9..c3b2dcb23dd32189f4985e19ca47277fce5a411d 100644 (file)
@@ -50,7 +50,6 @@
 #define SRVID_CTDB_PUSHDB  (CTDB_SRVID_TOOL_RANGE | 0x0002000000000000LL)
 
 static struct {
-       const char *socket;
        const char *debuglevelstr;
        int timelimit;
        int pnn;
@@ -6097,8 +6096,6 @@ static void usage(const char *command)
 
 struct poptOption cmdline_options[] = {
        POPT_AUTOHELP
-       { "socket", 's', POPT_ARG_STRING, &options.socket, 0,
-               "CTDB socket path", "filename" },
        { "debug", 'd', POPT_ARG_STRING, &options.debuglevelstr, 0,
                "debug level"},
        { "timelimit", 't', POPT_ARG_INT, &options.timelimit, 0,
@@ -6160,14 +6157,14 @@ static int process_command(const struct ctdb_cmd *cmd, int argc,
        }
 
        ctdb_socket = getenv("CTDB_SOCKET");
-       if (ctdb_socket != NULL) {
-               options.socket = ctdb_socket;
+       if (ctdb_socket == NULL) {
+               ctdb_socket = CTDB_SOCKET;
        }
 
-       ret = ctdb_client_init(ctdb, ctdb->ev, options.socket, &ctdb->client);
+       ret = ctdb_client_init(ctdb, ctdb->ev, ctdb_socket, &ctdb->client);
        if (ret != 0) {
                fprintf(stderr, "Failed to connect to CTDB daemon (%s)\n",
-                       options.socket);
+                       ctdb_socket);
 
                if (!find_node_xpnn(ctdb, NULL)) {
                        fprintf(stderr, "Is this node part of CTDB cluster?\n");
@@ -6231,7 +6228,6 @@ int main(int argc, const char *argv[])
        setlinebuf(stdout);
 
        /* Set default options */
-       options.socket = CTDB_SOCKET;
        options.debuglevelstr = NULL;
        options.timelimit = 10;
        options.sep = "|";