From a27b452fe7ba65ddba2617aa575ede81aec538c7 Mon Sep 17 00:00:00 2001
From: Zaiming Shi <zmstone@gmail.com>
Date: Tue, 9 Nov 2021 21:06:37 +0100
Subject: [PATCH 1/7] chore(bin/emqx): bash set -o

---
 bin/emqx | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/bin/emqx b/bin/emqx
index a97e13884..f8c4d3f6c 100755
--- a/bin/emqx
+++ b/bin/emqx
@@ -2,8 +2,7 @@
 # -*- tab-width:4;indent-tabs-mode:nil -*-
 # ex: ts=4 sw=4 et
 
-set -e
-set -o pipefail
+set -euo pipefail
 
 DEBUG="${DEBUG:-0}"
 if [ "$DEBUG" -eq 1 ]; then
@@ -171,11 +170,9 @@ if [ "$ES" -ne 0 ]; then
     exit $ES
 fi
 
-if [ -z "$WITH_EPMD" ]; then
-    EPMD_ARG="-start_epmd false -epmd_module ekka_epmd -proto_dist ekka"
-else
-    EPMD_ARG="-start_epmd true $PROTO_DIST_ARG"
-fi
+# EPMD_ARG="-start_epmd true $PROTO_DIST_ARG"
+NO_EPMD="-start_epmd false -epmd_module ekka_epmd -proto_dist ekka"
+EPMD_ARG="${EPMD_ARG:-${NO_EPMD}}"
 
 # Warn the user if ulimit -n is less than 1024
 ULIMIT_F=$(ulimit -n)

From 1480bb61585d9bb10a663bbc38f6b7f190d4d03b Mon Sep 17 00:00:00 2001
From: Zaiming Shi <zmstone@gmail.com>
Date: Tue, 9 Nov 2021 21:13:05 +0100
Subject: [PATCH 2/7] chore(bin/emqx): delete RELX_CONFIG_PATH

this variable is to allow setting sys.config by user
however this feature is now broken because the
we always generate app.<time>.config
---
 bin/emqx | 25 ++++---------------------
 1 file changed, 4 insertions(+), 21 deletions(-)

diff --git a/bin/emqx b/bin/emqx
index f8c4d3f6c..6ce1d8ad2 100755
--- a/bin/emqx
+++ b/bin/emqx
@@ -355,15 +355,6 @@ wait_for() {
     done
 }
 
-# Use $CWD/etc/sys.config if exists
-if [ -z "$RELX_CONFIG_PATH" ]; then
-    if [ -f "$RUNNER_ETC_DIR/sys.config" ]; then
-        RELX_CONFIG_PATH="-config $RUNNER_ETC_DIR/sys.config"
-    else
-        RELX_CONFIG_PATH=""
-    fi
-fi
-
 IS_BOOT_COMMAND='no'
 case "$1" in
     start|start_boot)
@@ -426,14 +417,6 @@ if [ -z "$COOKIE" ]; then
     die "Please set node.cookie in $RUNNER_ETC_DIR/emqx.conf or override from environment variable EMQX_NODE_COOKIE"
 fi
 
-# Support for IPv6 Dist. See: https://github.com/emqtt/emqttd/issues/1460
-PROTO_DIST="$(call_hocon -s $SCHEMA_MOD -I "$CONFIGS_DIR/" -c "$RUNNER_ETC_DIR"/emqx.conf get cluster.proto_dist | tr -d \")"
-if [ -z "$PROTO_DIST" ]; then
-    PROTO_DIST_ARG=""
-else
-    PROTO_DIST_ARG="-proto_dist $PROTO_DIST"
-fi
-
 cd "$ROOTDIR"
 
 case "$1" in
@@ -631,14 +614,14 @@ case "$1" in
         # Store passed arguments since they will be erased by `set`
         ARGS="$*"
 
-        # shellcheck disable=SC2086 # $RELX_CONFIG_PATH $CONFIG_ARGS $EPMD_ARG are supposed to be split by whitespace
+        # shellcheck disable=SC2086 # $CONFIG_ARGS $EPMD_ARG are supposed to be split by whitespace
         # Build an array of arguments to pass to exec later on
         # Build it here because this command will be used for logging.
         set -- "$BINDIR/erlexec" \
             -boot "$BOOTFILE" -mode "$CODE_LOADING_MODE" \
             -boot_var ERTS_LIB_DIR "$ERTS_LIB_DIR" \
             -mnesia dir "\"${MNESIA_DATA_DIR}\"" \
-            $RELX_CONFIG_PATH $CONFIG_ARGS $EPMD_ARG
+            $CONFIG_ARGS $EPMD_ARG
 
         # Log the startup
         logger -t "${REL_NAME}[$$]" "EXEC: $* -- ${1+$ARGS}"
@@ -672,14 +655,14 @@ case "$1" in
         # Store passed arguments since they will be erased by `set`
         ARGS="$*"
 
-        # shellcheck disable=SC2086 # $RELX_CONFIG_PATH $CONFIG_ARGS $EPMD_ARG are supposed to be split by whitespace
+        # shellcheck disable=SC2086 # $CONFIG_ARGS $EPMD_ARG are supposed to be split by whitespace
         # Build an array of arguments to pass to exec later on
         # Build it here because this command will be used for logging.
         set -- "$BINDIR/erlexec" $FOREGROUNDOPTIONS \
             -boot "$REL_DIR/$BOOTFILE" -mode "$CODE_LOADING_MODE" \
             -boot_var ERTS_LIB_DIR "$ERTS_LIB_DIR" \
             -mnesia dir "\"${MNESIA_DATA_DIR}\"" \
-            $RELX_CONFIG_PATH $CONFIG_ARGS $EPMD_ARG
+            $CONFIG_ARGS $EPMD_ARG
 
         # Log the startup
         logger -t "${REL_NAME}[$$]" "EXEC: $* -- ${1+$ARGS}"

From 9b9a070168f795c78074e56e6487be10bacebfc1 Mon Sep 17 00:00:00 2001
From: Zaiming Shi <zmstone@gmail.com>
Date: Tue, 9 Nov 2021 21:30:40 +0100
Subject: [PATCH 3/7] chore(bin/emqx): add default values for bash vars

---
 bin/emqx | 34 ++++++++++++++++++----------------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/bin/emqx b/bin/emqx
index 6ce1d8ad2..0266aba6b 100755
--- a/bin/emqx
+++ b/bin/emqx
@@ -225,7 +225,7 @@ relx_gen_id() {
 # Control a node
 relx_nodetool() {
     command="$1"; shift
-    ERL_FLAGS="$ERL_FLAGS $EPMD_ARG" \
+    ERL_FLAGS="${ERL_FLAGS:-} $EPMD_ARG" \
     "$ERTS_DIR/bin/escript" "$ROOTDIR/bin/nodetool" "$NAME_TYPE" "$NAME" \
                                 -setcookie "$COOKIE" "$command" "$@"
 }
@@ -356,7 +356,7 @@ wait_for() {
 }
 
 IS_BOOT_COMMAND='no'
-case "$1" in
+case "${1:-}" in
     start|start_boot)
         IS_BOOT_COMMAND='yes'
         ;;
@@ -366,6 +366,8 @@ case "$1" in
     foreground)
         IS_BOOT_COMMAND='yes'
         ;;
+    *)
+        ;;
 esac
 
 ## Possible ways to configure emqx node name:
@@ -419,7 +421,9 @@ fi
 
 cd "$ROOTDIR"
 
-case "$1" in
+COMMAND="${1:-}"
+
+case "${COMMAND}" in
     start|start_boot)
         # Make sure a node IS not running
         if relx_nodetool "ping" >/dev/null 2>&1; then
@@ -430,11 +434,9 @@ case "$1" in
 
         # this flag passes down to console mode
         # so we know it's intended to be run in daemon mode
-        export _EMQX_START_MODE="$1"
+        export _EMQX_START_MODE="$COMMAND"
 
-        # Save this for later.
-        CMD=$1
-        case "$1" in
+        case "$COMMAND" in
             start)
                 shift
                 START_OPTION="console"
@@ -453,7 +455,7 @@ case "$1" in
         [ "$RUN_PARAM" ] && set -- "$@" "$RUN_PARAM"
 
         # Export the HEART_COMMAND
-        HEART_COMMAND="$RUNNER_SCRIPT $CMD"
+        HEART_COMMAND="$RUNNER_SCRIPT $COMMAND"
         export HEART_COMMAND
 
         ## See: http://erlang.org/doc/man/run_erl.html
@@ -546,17 +548,17 @@ case "$1" in
         ;;
 
     upgrade|downgrade|install|unpack|uninstall)
-        if [ -z "$2" ]; then
+        if [ -z "${2:-}" ]; then
             echo "Missing version argument"
-            echo "Usage: $REL_NAME $1 {version}"
+            echo "Usage: $REL_NAME $COMMAND {version}"
             exit 1
         fi
 
-        COMMAND="$1"; shift
+        shift
 
         assert_node_alive
 
-        ERL_FLAGS="$ERL_FLAGS $EPMD_ARG" \
+        ERL_FLAGS="${ERL_FLAGS:-} $EPMD_ARG" \
         exec "$BINDIR/escript" "$ROOTDIR/bin/install_upgrade.escript" \
              "$COMMAND" "{'$REL_NAME', \"$NAME_TYPE\", '$NAME', '$COOKIE'}" "$@"
         ;;
@@ -564,9 +566,9 @@ case "$1" in
     versions)
         assert_node_alive
 
-        COMMAND="$1"; shift
+        shift
 
-        ERL_FLAGS="$ERL_FLAGS $EPMD_ARG" \
+        ERL_FLAGS="${ERL_FLAGS:-} $EPMD_ARG" \
         exec "$BINDIR/escript" "$ROOTDIR/bin/install_upgrade.escript" \
              "versions" "{'$REL_NAME', \"$NAME_TYPE\", '$NAME', '$COOKIE'}" "$@"
         ;;
@@ -578,7 +580,7 @@ case "$1" in
         # .boot file typically just $REL_NAME (ie, the app name)
         # however, for debugging, sometimes start_clean.boot is useful.
         # For e.g. 'setup', one may even want to name another boot script.
-        case "$1" in
+        case "$COMMAND" in
             console)
                 if [ -f "$REL_DIR/$REL_NAME.boot" ]; then
                   BOOTFILE="$REL_DIR/$REL_NAME"
@@ -707,7 +709,7 @@ case "$1" in
         relx_nodetool "eval" "$@"
         ;;
     *)
-        relx_usage "$1"
+        relx_usage "$COMMAND"
         exit 1
         ;;
 esac

From 25248a9e2178410d8d5f934936ee0fef93c988c2 Mon Sep 17 00:00:00 2001
From: Zaiming Shi <zmstone@gmail.com>
Date: Tue, 9 Nov 2021 21:34:37 +0100
Subject: [PATCH 4/7] chore(bin/emqx): delete boot file start option

we have never tested using alternative boot file to start EMQ X
drop it
---
 bin/emqx | 20 +++++---------------
 1 file changed, 5 insertions(+), 15 deletions(-)

diff --git a/bin/emqx b/bin/emqx
index 0266aba6b..5414ebcb2 100755
--- a/bin/emqx
+++ b/bin/emqx
@@ -139,7 +139,7 @@ relx_usage() {
             echo "                      don't make it permanent"
             ;;
         *)
-            echo "Usage: $REL_NAME {start|start_boot <file>|ertspath|foreground|stop|pid|ping|console|console_clean|console_boot <file>|attach|remote_console|upgrade|downgrade|install|uninstall|versions|escript|ctl|rpc|rpcterms|eval|root_dir}"
+            echo "Usage: $REL_NAME {start|ertspath|foreground|stop|pid|ping|console|console_clean|attach|remote_console|upgrade|downgrade|install|uninstall|versions|escript|ctl|rpc|rpcterms|eval|root_dir}"
             ;;
     esac
 }
@@ -357,10 +357,10 @@ wait_for() {
 
 IS_BOOT_COMMAND='no'
 case "${1:-}" in
-    start|start_boot)
+    start)
         IS_BOOT_COMMAND='yes'
         ;;
-    console|console_clean|console_boot)
+    console|console_clean)
         IS_BOOT_COMMAND='yes'
         ;;
     foreground)
@@ -424,7 +424,7 @@ cd "$ROOTDIR"
 COMMAND="${1:-}"
 
 case "${COMMAND}" in
-    start|start_boot)
+    start)
         # Make sure a node IS not running
         if relx_nodetool "ping" >/dev/null 2>&1; then
             die "node_is_already_running!"
@@ -442,11 +442,6 @@ case "${COMMAND}" in
                 START_OPTION="console"
                 HEART_OPTION="start"
                 ;;
-            start_boot)
-                shift
-                START_OPTION="console_boot"
-                HEART_OPTION="start_boot"
-                ;;
         esac
         RUN_PARAM="$*"
 
@@ -573,7 +568,7 @@ case "${COMMAND}" in
              "versions" "{'$REL_NAME', \"$NAME_TYPE\", '$NAME', '$COOKIE'}" "$@"
         ;;
 
-    console|console_clean|console_boot)
+    console|console_clean)
         # Bootstrap daemon command (check perms & drop to $RUNNER_USER)
         bootstrapd
 
@@ -591,11 +586,6 @@ case "${COMMAND}" in
             console_clean)
                 BOOTFILE="$REL_DIR/start_clean"
                 ;;
-            console_boot)
-                shift
-                BOOTFILE="$1"
-                shift
-                ;;
         esac
 
         # set before generate_config

From cf06ceb9211f5020512c204ef463c0a582295a14 Mon Sep 17 00:00:00 2001
From: Zaiming Shi <zmstone@gmail.com>
Date: Tue, 9 Nov 2021 21:37:06 +0100
Subject: [PATCH 5/7] chore(bin/emqx): fix a typo in bash function

---
 bin/emqx | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/bin/emqx b/bin/emqx
index 5414ebcb2..0775b5f84 100755
--- a/bin/emqx
+++ b/bin/emqx
@@ -65,15 +65,15 @@ assert_node_alive() {
 # Echo to stderr on errors
 echoerr() { echo "$*" 1>&2; }
 
-check_eralng_start() {
+check_erlang_start() {
     "$BINDIR/$PROGNAME" -noshell -boot "$REL_DIR/start_clean" -s crypto start -s init stop
 }
 
-if ! check_eralng_start >/dev/null 2>&1; then
+if ! check_erlang_start >/dev/null 2>&1; then
     BUILT_ON="$(head -1 "${REL_DIR}/BUILT_ON")"
     ## failed to start, might be due to missing libs, try to be portable
     export LD_LIBRARY_PATH="$DYNLIBS_DIR:$LD_LIBRARY_PATH"
-    if ! check_eralng_start; then
+    if ! check_erlang_start; then
         ## it's hopeless
         echoerr "FATAL: Unable to start Erlang (with libcrypto)."
         echoerr "Please make sure it's running on the correct platform with all required dependencies."

From 160825095a717aa53b1906bb64389f54dbc72a6e Mon Sep 17 00:00:00 2001
From: Zaiming Shi <zmstone@gmail.com>
Date: Tue, 9 Nov 2021 22:58:15 +0100
Subject: [PATCH 6/7] chore(bin/emqx): add more usage prints

---
 bin/emqx | 211 ++++++++++++++++++++++++++++++++++++-------------------
 1 file changed, 140 insertions(+), 71 deletions(-)

diff --git a/bin/emqx b/bin/emqx
index 0775b5f84..fb62aac0c 100755
--- a/bin/emqx
+++ b/bin/emqx
@@ -66,9 +66,141 @@ assert_node_alive() {
 echoerr() { echo "$*" 1>&2; }
 
 check_erlang_start() {
-    "$BINDIR/$PROGNAME" -noshell -boot "$REL_DIR/start_clean" -s crypto start -s init stop
+    "$BINDIR/$PROGNAME" -noshell -boot "$REL_DIR/start_clean" -s crypto start -s erlang halt
 }
 
+usage() {
+    local command="$1"
+
+    case "$command" in
+    start)
+        echo "Start EMQ X service in daemon mode"
+        ;;
+    stop)
+        echo "Stop the running EMQ X program"
+        ;;
+    restart|reboot)
+        echo "Restart $EMQX_DESCRIPTION"
+        ;;
+    pid)
+        echo "Print out $EMQX_DESCRIPTION process identifier"
+        ;;
+    ping)
+        echo "Check if the $EMQX_DESCRIPTION node is up and running"
+        echo "This command exit with 0 silently if node is running"
+        ;;
+    escript)
+        echo "Execute a escript using the Erlang runtime from EMQ X package installation"
+        echo "For example $REL_NAME escript /path/to/my/escript my_arg1 my_arg2"
+        ;;
+    attach)
+        echo "This command is applicable when $EMQX_DESCRIPTION is started in daemon"
+        echo "mode. it attaches the current shell to EMQ X's control console"
+        echo "through a named pipe"
+        echo "WARNING: try to use the safer alternative, remote_console command."
+        ;;
+    remote_console)
+        echo "Start a dummy Erlang node and hidden-connect $EMQX_DESCRIPTION to"
+        echo "with an interactive Erlang shell"
+        ;;
+    console)
+        echo "Boot up $EMQX_DESCRIPTION service in an interactive Erlang shell"
+        echo "This command is useful for troubleshooting"
+        ;;
+    console_clean)
+        echo "This command does NOT boot up the $EMQX_DESCRIPTION service"
+        echo "It only starts an interactive Erlang console with all the"
+        echo "EMQ X code available"
+        ;;
+    foreground)
+        echo "Start $EMQX_DESCRIPTION in foreground mode"
+        ;;
+    ertspath)
+        echo "Print path to Erlang runtime dir"
+        ;;
+    rpc)
+        echo "Usge $REL_NAME rpc MODULE FUNCTION [ARGS, ...]"
+        echo "Connect to the $EMQX_DESCRIPTION node and make an Erlang RPC"
+        echo "The result of the RPC call must be 'ok'"
+        echo "This command blocks for at most 60 seconds in case the node"
+        echo "does not reply the call in time"
+        ;;
+    rpcterms)
+        echo "Usge $REL_NAME rpcterms MODULE FUNCTION [ARGS, ...]"
+        echo "Connect to the $EMQX_DESCRIPTION node and make an Erlang RPC"
+        echo "The result of the RPC call is pretty-printed as an Erlang term"
+        ;;
+    root_dir)
+        echo "Print EMQ X installation root dir"
+        ;;
+    eval)
+        echo "Evaluate an Erlang expression in the EMQ X node"
+        ;;
+    versions)
+        echo "List installed EMQ X versions and their status"
+        ;;
+    unpack)
+        echo "Usage: $REL_NAME unpack [VERSION]"
+        echo "Unpacks a release package VERSION, it assumes that this"
+        echo "release package tarball has already been deployed at one"
+        echo "of the following locations:"
+        echo "      releases/<relname>-<version>.tar.gz"
+        echo "      releases/<relname>-<version>.zip"
+        ;;
+    install)
+        echo "Usage: $REL_NAME install [VERSION]"
+        echo "Installs a release package VERSION, it assumes that this"
+        echo "release package tarball has already been deployed at one"
+        echo "of the following locations:"
+        echo "      releases/<relname>-<version>.tar.gz"
+        echo "      releases/<relname>-<version>.zip"
+        echo ""
+        echo "     --no-permanent   Install release package VERSION but"
+        echo "                      don't make it permanent"
+        ;;
+    uninstall)
+        echo "Usage: $REL_NAME uninstall [VERSION]"
+        echo "Uninstalls a release VERSION, it will only accept"
+        echo "versions that are not currently in use"
+        ;;
+    upgrade)
+        echo "Usage: $REL_NAME upgrade [VERSION]"
+        echo "Upgrades the currently running release to VERSION, it assumes"
+        echo "that a release package tarball has already been deployed at one"
+        echo "of the following locations:"
+        echo "      releases/<relname>-<version>.tar.gz"
+        echo "      releases/<relname>-<version>.zip"
+        echo ""
+        echo "     --no-permanent   Install release package VERSION but"
+        echo "                      don't make it permanent"
+        ;;
+    downgrade)
+        echo "Usage: $REL_NAME downgrade [VERSION]"
+        echo "Downgrades the currently running release to VERSION, it assumes"
+        echo "that a release package tarball has already been deployed at one"
+        echo "of the following locations:"
+        echo "      releases/<relname>-<version>.tar.gz"
+        echo "      releases/<relname>-<version>.zip"
+        echo ""
+        echo "     --no-permanent   Install release package VERSION but"
+        echo "                      don't make it permanent"
+        ;;
+    *)
+        echo "Usage: $REL_NAME {start|ertspath|foreground|stop|pid|ping|console|console_clean|attach|remote_console|upgrade|downgrade|install|uninstall|versions|escript|ctl|rpc|rpcterms|eval|root_dir} <help>"
+    ;;
+    esac
+}
+
+COMMAND="${1:-}"
+
+if [ "${2:-}" = 'help' ]; then
+    ## 'ctl' command has its own usage info
+    if [ "$COMMAND" != 'ctl' ]; then
+        usage "$COMMAND"
+        exit 0
+    fi
+fi
+
 if ! check_erlang_start >/dev/null 2>&1; then
     BUILT_ON="$(head -1 "${REL_DIR}/BUILT_ON")"
     ## failed to start, might be due to missing libs, try to be portable
@@ -88,62 +220,6 @@ if [ -d "$ERTS_DIR/lib" ]; then
     export LD_LIBRARY_PATH="$ERTS_DIR/lib:$LD_LIBRARY_PATH"
 fi
 
-relx_usage() {
-    command="$1"
-
-    case "$command" in
-        unpack)
-            echo "Usage: $REL_NAME unpack [VERSION]"
-            echo "Unpacks a release package VERSION, it assumes that this"
-            echo "release package tarball has already been deployed at one"
-            echo "of the following locations:"
-            echo "      releases/<relname>-<version>.tar.gz"
-            echo "      releases/<relname>-<version>.zip"
-            ;;
-        install)
-            echo "Usage: $REL_NAME install [VERSION]"
-            echo "Installs a release package VERSION, it assumes that this"
-            echo "release package tarball has already been deployed at one"
-            echo "of the following locations:"
-            echo "      releases/<relname>-<version>.tar.gz"
-            echo "      releases/<relname>-<version>.zip"
-            echo ""
-            echo "     --no-permanent   Install release package VERSION but"
-            echo "                      don't make it permanent"
-            ;;
-        uninstall)
-            echo "Usage: $REL_NAME uninstall [VERSION]"
-            echo "Uninstalls a release VERSION, it will only accept"
-            echo "versions that are not currently in use"
-            ;;
-        upgrade)
-            echo "Usage: $REL_NAME upgrade [VERSION]"
-            echo "Upgrades the currently running release to VERSION, it assumes"
-            echo "that a release package tarball has already been deployed at one"
-            echo "of the following locations:"
-            echo "      releases/<relname>-<version>.tar.gz"
-            echo "      releases/<relname>-<version>.zip"
-            echo ""
-            echo "     --no-permanent   Install release package VERSION but"
-            echo "                      don't make it permanent"
-            ;;
-        downgrade)
-            echo "Usage: $REL_NAME downgrade [VERSION]"
-            echo "Downgrades the currently running release to VERSION, it assumes"
-            echo "that a release package tarball has already been deployed at one"
-            echo "of the following locations:"
-            echo "      releases/<relname>-<version>.tar.gz"
-            echo "      releases/<relname>-<version>.zip"
-            echo ""
-            echo "     --no-permanent   Install release package VERSION but"
-            echo "                      don't make it permanent"
-            ;;
-        *)
-            echo "Usage: $REL_NAME {start|ertspath|foreground|stop|pid|ping|console|console_clean|attach|remote_console|upgrade|downgrade|install|uninstall|versions|escript|ctl|rpc|rpcterms|eval|root_dir}"
-            ;;
-    esac
-}
-
 # Simple way to check the correct user and fail early
 check_user() {
     # Validate that the user running the script is the owner of the
@@ -355,18 +431,13 @@ wait_for() {
     done
 }
 
-IS_BOOT_COMMAND='no'
-case "${1:-}" in
-    start)
-        IS_BOOT_COMMAND='yes'
-        ;;
-    console|console_clean)
-        IS_BOOT_COMMAND='yes'
-        ;;
-    foreground)
+## IS_BOOT_COMMAND is set for later to inspect node name and cookie from hocon config (or env variable)
+case "${COMMAND}" in
+    start|console|console_clean|foreground)
         IS_BOOT_COMMAND='yes'
         ;;
     *)
+        IS_BOOT_COMMAND='no'
         ;;
 esac
 
@@ -421,8 +492,6 @@ fi
 
 cd "$ROOTDIR"
 
-COMMAND="${1:-}"
-
 case "${COMMAND}" in
     start)
         # Make sure a node IS not running
@@ -500,8 +569,8 @@ case "${COMMAND}" in
         ;;
 
     restart|reboot)
-        echo "$EMQX_DESCRIPTION $REL_VSN is stopped: $("$RUNNER_BIN_DIR"/emqx stop)"
-        "$RUNNER_BIN_DIR"/emqx start
+        echo "$EMQX_DESCRIPTION $REL_VSN is stopped: $("$RUNNER_BIN_DIR/$REL_NAME" stop)"
+        "$RUNNER_BIN_DIR/$REL_NAME" start
         ;;
 
     pid)
@@ -699,7 +768,7 @@ case "${COMMAND}" in
         relx_nodetool "eval" "$@"
         ;;
     *)
-        relx_usage "$COMMAND"
+        usage "$COMMAND"
         exit 1
         ;;
 esac

From e484fcc8c370898eda8fedfaface3046a932a3d2 Mon Sep 17 00:00:00 2001
From: Zaiming Shi <zmstone@gmail.com>
Date: Wed, 10 Nov 2021 21:13:39 +0100
Subject: [PATCH 7/7] chore(bin/emqx): write sensible log if node not
 initialized

---
 bin/emqx | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/bin/emqx b/bin/emqx
index fb62aac0c..070f4bd7f 100755
--- a/bin/emqx
+++ b/bin/emqx
@@ -431,6 +431,22 @@ wait_for() {
     done
 }
 
+latest_vm_args() {
+    local hint_var_name="$1"
+    local vm_args_file
+    vm_args_file="$(find "$CONFIGS_DIR" -type f -name "vm.*.args" | sort | tail -1)"
+    if [ -f "$vm_args_file" ]; then
+        echo "$vm_args_file"
+    else
+        echoerr "ERRRO: node not initialized?"
+        echoerr "Generated config file vm.*.args is not found for command '$COMMAND'"
+        echoerr "in config dir: $CONFIGS_DIR"
+        echoerr "In case the file has been deleted while the node is running,"
+        echoerr "set environment variable '$hint_var_name' to continue"
+        exit 1
+    fi
+}
+
 ## IS_BOOT_COMMAND is set for later to inspect node name and cookie from hocon config (or env variable)
 case "${COMMAND}" in
     start|console|console_clean|foreground)
@@ -452,10 +468,8 @@ if [ -z "$NAME" ]; then
         # for boot commands, inspect emqx.conf for node name
         NAME="$(call_hocon -s $SCHEMA_MOD -I "$CONFIGS_DIR/" -c "$RUNNER_ETC_DIR"/emqx.conf get node.name | tr -d \")"
     else
-        # for non-boot commands, inspect vm.<time>.args for node name
-        # shellcheck disable=SC2012,SC2086
-        LATEST_VM_ARGS="$(ls -t $CONFIGS_DIR/vm.*.args | head -1)"
-        NAME="$(grep -E '^-s?name' "$LATEST_VM_ARGS" | awk '{print $2}')"
+        vm_args_file="$(latest_vm_args 'EMQX_NODE_NAME')"
+        NAME="$(grep -E '^-s?name' "${vm_args_file}" | awk '{print $2}')"
     fi
 fi
 
@@ -480,9 +494,8 @@ if [ -z "$COOKIE" ]; then
     if [ "$IS_BOOT_COMMAND" = 'yes' ]; then
         COOKIE="$(call_hocon -s $SCHEMA_MOD -I "$CONFIGS_DIR/" -c "$RUNNER_ETC_DIR"/emqx.conf get node.cookie | tr -d \")"
     else
-        # shellcheck disable=SC2012,SC2086
-        LATEST_VM_ARGS="$(ls -t $CONFIGS_DIR/vm.*.args | head -1)"
-        COOKIE="$(grep -E '^-setcookie' "$LATEST_VM_ARGS" | awk '{print $2}')"
+        vm_args_file="$(latest_vm_args 'EMQX_NODE_COOKIE')"
+        COOKIE="$(grep -E '^-setcookie' "${vm_args_file}" | awk '{print $2}')"
     fi
 fi