airship/shipyard
Code Issues Proposed changes

tools: fix execute_shipyard_action if condition

Browse Source 
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
This commit is contained in:
Dustin Specker 2019-02-22 10:48:10 -06:00 committed by Kaspars Skels
parent 2d10945dd8
commit f4f57a1bbf
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/execute_shipyard_action.sh
View File

@ -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}"