From 343d649725bf392acab403833223c263520ac7f6 Mon Sep 17 00:00:00 2001
From: Evgeny L <eli@mirantis.com>
Date: Wed, 24 Apr 2019 18:46:31 +0000
Subject: [PATCH] Fix shipyard subcommand for airship cli

1. Docker env file format does not support quotation marks [0].
2. Use `shift` to remove `shipyard` command from the command,
   shipyard image already has it configured in the entrypoint
   script.

[0] See the official docs https://docs.docker.com/compose/env-file/

Change-Id: I8deaa60355004e182f51872a2fd15d94297ff319
---
 tools/airship | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/tools/airship b/tools/airship
index e8f04bcdb..000980201 100755
--- a/tools/airship
+++ b/tools/airship
@@ -11,16 +11,16 @@ trap "{ rm -f $ENV_FILE; }" EXIT
 
 # prepare docker environment file
 cat > $ENV_FILE << EOF
-PEGLEG_PASSPHRASE=${PEGLEG_PASSPHRASE:-'password123'}
-PEGLEG_SALT=${PEGLEG_SALT:=-'password123'}
+PEGLEG_PASSPHRASE=${PEGLEG_PASSPHRASE:-password123}
+PEGLEG_SALT=${PEGLEG_SALT:-password123}
 
-OS_AUTH_URL=${OS_AUTH_URL:-'http://keystone-api.ucp.svc.cluster.local:5000/v3'}
-OS_PASSWORD=${OS_PASSWORD:-'password123'}
-OS_PROJECT_DOMAIN_NAME=${OS_PROJECT_DOMAIN_NAME:-'default'}
-OS_PROJECT_NAME=${OS_PROJECT_NAME:-'service'}
-OS_USERNAME=${OS_USERNAME:-'shipyard'}
-OS_USER_DOMAIN_NAME=${OS_USER_DOMAIN_NAME:-'default'}
-OS_IDENTITY_API_VERSION=${OS_IDENTITY_API_VERSION:-'3'}
+OS_AUTH_URL=${OS_AUTH_URL:-http://keystone-api.ucp.svc.cluster.local:5000/v3}
+OS_PASSWORD=${OS_PASSWORD:-password123}
+OS_PROJECT_DOMAIN_NAME=${OS_PROJECT_DOMAIN_NAME:-default}
+OS_PROJECT_NAME=${OS_PROJECT_NAME:-service}
+OS_USERNAME=${OS_USERNAME:-shipyard}
+OS_USER_DOMAIN_NAME=${OS_USER_DOMAIN_NAME:-default}
+OS_IDENTITY_API_VERSION=${OS_IDENTITY_API_VERSION:-3}
 EOF
 
 REPO_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/../ >/dev/null 2>&1 && pwd )"
@@ -142,6 +142,7 @@ case "$1" in
   promenade $@
   ;;
 'shipyard')
+  shift;
   shipyard $@
   ;;
 *) help