Used set to make shell scripts more strict

dev-tools/signoff-check.sh

@@ -1,5 +1,7 @@
 #!/bin/sh
 
+set -e
+
 ### Script to check for signoff presents on commits
 
 # Validate input parameters

distribution/docker/docker-test-entrypoint.sh

@@ -1,4 +1,6 @@
 #!/usr/bin/env bash
+set -e
+
 cd /usr/share/opensearch/bin/
 
 /usr/local/bin/docker-entrypoint.sh | tee > /usr/share/opensearch/logs/console.log

distribution/packages/src/common/scripts/preinst

@@ -10,6 +10,8 @@
 #       $1=1       : indicates an new install
 #       $1=2       : indicates an upgrade
 
+set -e
+
 err_exit() {
     echo "$@" >&2
     exit 1

distribution/packages/src/common/systemd/systemd-entrypoint

@@ -3,6 +3,8 @@
 # This wrapper script allows SystemD to feed a file containing a passphrase into
 # the main OpenSearch startup script
 
+set -e
+
 if [ -n "$OPENSEARCH_KEYSTORE_PASSPHRASE_FILE" ] ; then
   exec /usr/share/opensearch/bin/opensearch "$@" < "$OPENSEARCH_KEYSTORE_PASSPHRASE_FILE"
 else

distribution/packages/src/deb/init.d/opensearch

@@ -12,6 +12,8 @@
 # Description:       Starts opensearch using start-stop-daemon
 ### END INIT INFO
 
+set -e
+
 PATH=/bin:/usr/bin:/sbin:/usr/sbin
 NAME=opensearch
 DESC="OpenSearch Server"

distribution/packages/src/rpm/init.d/opensearch

@@ -16,6 +16,8 @@
 # Description: OpenSearch is a very scalable, schema-free and high-performance search solution supporting multi-tenancy and near realtime search.
 ### END INIT INFO
 
+set -e
+
 #
 # init.d / servicectl compatibility (openSUSE)
 #

distribution/src/bin/opensearch

@@ -13,6 +13,8 @@
 #
 #   OPENSEARCH_JAVA_OPTS="-Xms8g -Xmx8g" ./bin/opensearch
 
+set -e
+
 source "`dirname "$0"`"/opensearch-env
 
 CHECK_KEYSTORE=true

distribution/src/bin/opensearch-cli

@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 
-set -e -o pipefail
+set -e
 
 source "`dirname "$0"`"/opensearch-env
 

distribution/src/bin/opensearch-env

@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 
-set -e -o pipefail
+set -e
 
 CDPATH=""
 

distribution/src/bin/opensearch-env-from-file

@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 
-set -e -o pipefail
+set -e
 
 # Allow environment variables to be set by creating a file with the
 # contents, and setting an environment variable with the suffix _FILE to

distribution/src/bin/opensearch-keystore

@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+set -e
+
 OPENSEARCH_MAIN_CLASS=org.opensearch.common.settings.KeyStoreCli \
   OPENSEARCH_ADDITIONAL_CLASSPATH_DIRECTORIES=lib/tools/keystore-cli \
   "`dirname "$0"`"/opensearch-cli \

distribution/src/bin/opensearch-node

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -e
 
 OPENSEARCH_MAIN_CLASS=org.opensearch.cluster.coordination.NodeToolCli \
   "`dirname "$0"`"/opensearch-cli \

distribution/src/bin/opensearch-plugin

@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+set -e
+
 OPENSEARCH_MAIN_CLASS=org.opensearch.plugins.PluginCli \
   OPENSEARCH_ADDITIONAL_CLASSPATH_DIRECTORIES=lib/tools/plugin-cli \
   "`dirname "$0"`"/opensearch-cli \

distribution/src/bin/opensearch-shard

@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+set -e
+
 OPENSEARCH_MAIN_CLASS=org.opensearch.index.shard.ShardToolCli \
   "`dirname "$0"`"/opensearch-cli \
   "$@"

distribution/src/bin/opensearch-upgrade

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -e
 
 OPENSEARCH_MAIN_CLASS=org.opensearch.upgrade.UpgradeCli \
   OPENSEARCH_ADDITIONAL_CLASSPATH_DIRECTORIES=lib/tools/upgrade-cli \

qa/remote-clusters/docker-test-entrypoint.sh

@@ -1,4 +1,6 @@
 #!/usr/bin/env bash
+set -e
+
 cd /usr/share/opensearch/bin/
 ./opensearch-users useradd rest_user -p test-password -r superuser || true
 echo "testnode" > /tmp/password