From f4f57a1bbf3cc9ca6c868a11cc8923326c81b6dc Mon Sep 17 00:00:00 2001
From: Dustin Specker <dustin.specker@att.com>
Date: Fri, 22 Feb 2019 10:48:10 -0600
Subject: [PATCH] tools: fix execute_shipyard_action if condition

Before if condition was looking for non-existence of ${server}. `server`
is never defined, so this condition is always true.

Now if condition is looking for existence of ${servers}, which is
sometimes defined.

Change-Id: I31cd4a2765705176a9d5239a01b04ff341b36540
---
 tools/execute_shipyard_action.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/execute_shipyard_action.sh b/tools/execute_shipyard_action.sh
index 6ad4db06..024fb248 100755
--- a/tools/execute_shipyard_action.sh
+++ b/tools/execute_shipyard_action.sh
@@ -50,7 +50,7 @@ run_action () {
     # Note that deploy and update site do not require additional parameter
     # to be passed in while redeploy_server requires user to indicate which
     # servers to redeploy
-    if ! [[ ${server} ]] && [[ ${action} ]]; then
+    if ! [[ ${servers} ]] && [[ ${action} ]]; then
         ${base_docker_command} ${SHIPYARD_IMAGE} create action ${action}
     elif [[ ${action} == 'redeploy_server' && ${servers} ]]; then
         ${base_docker_command} ${SHIPYARD_IMAGE} create action redeploy_server --param="target_nodes=${servers}"