Merge pull request #422 from QuentinPerez/fix-ps

fix: ps --filter

Author: Quentin Perez

README.md

@@ -1201,6 +1201,10 @@ $ scw inspect myserver | jq '.[0].public_ip.address'
 
 ## Changelog
 
+### master (unreleased)
+
+* `scw ps` do not display empty server with --filter
+
 ### v1.10.1 (2016-10-24)
 
 * `scw login` fix CheckCredentials ([418](https://github.com/scaleway/scaleway-cli/issues/418))

pkg/commands/ps.go

@@ -53,8 +53,7 @@ func RunPs(ctx CommandContext, args PsArgs) error {
 			logrus.Warnf("Unknown filter: '%s=%s'", key, value)
 		}
 	}
-	filtered := make([]api.ScalewayServer, len(*servers))
-	index := 0
+	filtered := make([]api.ScalewayServer, 0, len(*servers))
 	for _, server := range *servers {
 		// filtering
 		for key, value := range args.Filters {
@@ -100,8 +99,7 @@ func RunPs(ctx CommandContext, args PsArgs) error {
 				}
 			}
 		}
-		filtered[index] = server
-		index++
+		filtered = append(filtered, server)
 	skipServer:
 		continue
 	}