Return empty list on error

French Ben · French Ben · commit 20e050604f0f · 2016-11-21T11:47:02.000-08:00
Signed-off-by: French Ben &lt;frenchben@docker.com&gt;
diff --git a/tools/metaserver/src/metaserver/azure.go b/tools/metaserver/src/metaserver/azure.go
@@ -92,6 +92,7 @@ func (a AzureWeb) Managers() []WebInstance {
 	managerVMs, err := getVMSSList(vmssClient, env, env["AZURE_VMSS_MGR"], managerIPTable)
 	if err != nil {
 		fmt.Printf("Couldn't get List of Manager VMSS: %v", err)
+		return []WebInstance{}
 	}
 	return managerVMs
 }
@@ -118,6 +119,7 @@ func (a AzureWeb) Workers() []WebInstance {
 	workerVMs, err := getVMSSList(vmssClient, env, env["AZURE_VMSS_WRK"], workerIPTable)
 	if err != nil {
 		fmt.Printf("Couldn't get List of Worker VMSS: %v", err)
+		return []WebInstance{}
 	}
 	return workerVMs
 }

Original file line number	Diff line number	Diff line change
`@@ -92,6 +92,7 @@ func (a AzureWeb) Managers() []WebInstance {`
`92`	`92`	`managerVMs, err := getVMSSList(vmssClient, env, env["AZURE_VMSS_MGR"], managerIPTable)`
`93`	`93`	`if err != nil {`
`94`	`94`	`fmt.Printf("Couldn't get List of Manager VMSS: %v", err)`
	`95`	`+ return []WebInstance{}`
`95`	`96`	`}`
`96`	`97`	`return managerVMs`
`97`	`98`	`}`
`@@ -118,6 +119,7 @@ func (a AzureWeb) Workers() []WebInstance {`
`118`	`119`	`workerVMs, err := getVMSSList(vmssClient, env, env["AZURE_VMSS_WRK"], workerIPTable)`
`119`	`120`	`if err != nil {`
`120`	`121`	`fmt.Printf("Couldn't get List of Worker VMSS: %v", err)`
	`122`	`+ return []WebInstance{}`
`121`	`123`	`}`
`122`	`124`	`return workerVMs`
`123`	`125`	`}`