Fixed multiple-instances case where the input is empty

outbrain · Nov 23, 2015 · 79fde52 · 79fde52
1 parent 6214b79
commit 79fde52
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/build.sh b/build.sh
@@ -6,7 +6,7 @@
 #
 set -e
 
-RELEASE_VERSION="1.4.535"
+RELEASE_VERSION="1.4.536"
 TOPDIR=/tmp/orchestrator-release
 export RELEASE_VERSION TOPDIR
 

diff --git a/go/app/cli.go b/go/app/cli.go
@@ -121,8 +121,9 @@ func deduceInstanceKeyIfNeeded(instance string, instanceKey *inst.InstanceKey) *
 // to take multiple instance names separated by a comma or whitespace.
 func CliWrapper(command string, strict bool, instances string, destination string, owner string, reason string, duration string, pattern string, clusterAlias string, pool string, hostnameFlag string) {
 	r := regexp.MustCompile(`[ ,\r\n\t]+`)
-	for _, instance := range r.Split(instances, -1) {
-		if instance != "" {
+	tokens := r.Split(instances, -1)
+	for _, instance := range tokens {
+		if instance != "" || len(tokens) == 1 {
 			Cli(command, strict, instance, destination, owner, reason, duration, pattern, clusterAlias, pool, hostnameFlag)
 		}
 	}