Fix arg array ordering and rename network-name option

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

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 \