From 50444181c5c49c6221f9ef1ea084dfbcdafa2f23 Mon Sep 17 00:00:00 2001
From: Dan Albert <dan@solana.com>
Date: Fri, 5 Apr 2019 11:45:28 -0600
Subject: [PATCH] Fix arg array ordering and rename network-name option

---
 ci/testnet-deploy.sh  | 14 +++++++-------
 ci/testnet-manager.sh |  4 ++--
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/ci/testnet-deploy.sh b/ci/testnet-deploy.sh
index da6878538e..272c97b7e4 100755
--- a/ci/testnet-deploy.sh
+++ b/ci/testnet-deploy.sh
@@ -27,12 +27,12 @@ usage() {
     echo "Error: $*"
   fi
   cat <<EOF
-usage: $0 -N name -C cloud -z zone1 [-z zone2] ... [-z zoneN] [options...]
+usage: $0 -p network-name -C cloud -z zone1 [-z zone2] ... [-z zoneN] [options...]
 
 Deploys a CD testnet
 
   mandatory arguments:
-  -N [name]  - name of the network
+  -p [network-name]  - name of the network
   -C [cloud] - cloud provider to use (gce, ec2)
   -z [zone]  - cloud provider zone to deploy the network into.  Must specify at least one zone
 
@@ -62,12 +62,12 @@ EOF
 
 zone=()
 
-while getopts "h?p:Pn:c:t:gG:a:Dbd:rusxz:N:C:" opt; do
+while getopts "h?p:Pn:c:t:gG:a:Dbd:rusxz:p:C:" opt; do
   case $opt in
   h | \?)
     usage
     ;;
-  N)
+  p)
     netName=$OPTARG
     ;;
   C)
@@ -158,7 +158,7 @@ set -x
 
 # Build a string to pass zone opts to $cloudProvider.sh: "-z zone1 -z zone2 ..."
 for val in "${zone[@]}"; do
-  zone_args="-z $zone_args $val"
+  zone_args="$zone_args -z $val"
 done
 
 if ! $skipSetup; then
@@ -175,7 +175,7 @@ if ! $skipSetup; then
     -c "$clientNodeCount"
     -n "$additionalFullNodeCount"
   )
-  create_args+=("$zone_args ")
+  create_args+=("$zone_args")
 
   if $blockstreamer; then
     create_args+=(-u)
@@ -211,7 +211,7 @@ else
   config_args=(
     -p "$netName"
   )
-  config_args+=("$zone_args ")
+  config_args+=("$zone_args")
   if $publicNetwork; then
     config_args+=(-P)
   fi
diff --git a/ci/testnet-manager.sh b/ci/testnet-manager.sh
index f05c8523cd..f0bda90ee3 100755
--- a/ci/testnet-manager.sh
+++ b/ci/testnet-manager.sh
@@ -230,11 +230,11 @@ start() {
 
       # Build a string to pass as opts to testnet-deploy.sh: "-z zone1 -z zone2 ..."
       for val in "${GCE_ZONES[@]}"; do
-        GCE_ZONE_ARGS="-z $GCE_ZONE_ARGS $val"
+        GCE_ZONE_ARGS="$GCE_ZONE_ARGS -z $val"
       done
 
       for val in "${EC2_ZONES[@]}"; do
-        EC2_ZONE_ARGS="-z $EC2_ZONE_ARGS $val"
+        EC2_ZONE_ARGS="$EC2_ZONE_ARGS -z $val"
       done
 
       NO_VALIDATOR_SANITY=1 \