From 3e165869b1dc901a2b7dd05ac8dac27fe59d2a42 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Mon, 23 Apr 2018 06:56:17 -0400 Subject: [PATCH] refactor: allow usage of the same variable to disable docker pulls --- config | 2 ++ functions | 4 ++-- install | 4 ++-- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/config b/config index fbdefc8..c7dcddf 100644 --- a/config +++ b/config @@ -11,6 +11,8 @@ export PLUGIN_DATA_HOST_ROOT=$ELASTICSEARCH_HOST_ROOT export PLUGIN_DATASTORE_PORTS=(9200 9300) export PLUGIN_DATASTORE_WAIT_PORT=9200 export PLUGIN_DEFAULT_ALIAS="ELASTICSEARCH" +export PLUGIN_DISABLE_PULL=${ELASTICSEARCH_DISABLE_PULL:=} +export PLUGIN_DISABLE_PULL_VARIABLE="ELASTICSEARCH_DISABLE_PULL" export PLUGIN_ALT_ALIAS="DOKKU_ELASTICSEARCH" export PLUGIN_IMAGE=$ELASTICSEARCH_IMAGE export PLUGIN_IMAGE_VERSION=$ELASTICSEARCH_IMAGE_VERSION diff --git a/functions b/functions index 4585ef7..a136079 100755 --- a/functions +++ b/functions @@ -27,8 +27,8 @@ service_create() { service_parse_args "${@:2}" if ! docker images | grep -e "^$PLUGIN_IMAGE " | grep -q " $PLUGIN_IMAGE_VERSION " ; then - if [[ "$ELASTICSEARCH_DISABLE_PULL" == "true" ]]; then - dokku_log_warn "ELASTICSEARCH_DISABLE_PULL environment variable detected. Not running pull command." 1>&2 + if [[ "$PLUGIN_DISABLE_PULL" == "true" ]]; then + dokku_log_warn "${PLUGIN_DISABLE_PULL_VARIABLE} environment variable detected. Not running pull command." 1>&2 dokku_log_warn " docker pull ${IMAGE}" 1>&2 dokku_log_warn "$PLUGIN_SERVICE service creation failed" exit 1 diff --git a/install b/install index 82513c4..a5cf182 100755 --- a/install +++ b/install @@ -5,8 +5,8 @@ set -eo pipefail; [[ $DOKKU_TRACE ]] && set -x plugin-install() { pull-docker-image() { declare IMAGE="$1" - if [[ "$ELASTICSEARCH_DISABLE_PULL" == "true" ]]; then - echo " ! ELASTICSEARCH_DISABLE_PULL environment variable detected. Not running pull command." 1>&2 + if [[ "$PLUGIN_DISABLE_PULL" == "true" ]]; then + echo " ! ${PLUGIN_DISABLE_PULL_VARIABLE} environment variable detected. Not running pull command." 1>&2 echo " ! docker pull ${IMAGE}" 1>&2 return fi