Merge pull request #224 from arianvp/start-server-bug

moul · moul · commit e8825d42f806 · 2015-10-19T10:48:42.000+02:00
Fix bug that starting an already started server succeeds.
diff --git a/pkg/api/helpers.go b/pkg/api/helpers.go
@@ -485,7 +485,7 @@ func StartServer(api *ScalewayAPI, needle string, wait bool) error {
 
 	err := api.PostServerAction(server, "poweron")
 	if err != nil {
-		if err.Error() != "server should be stopped" {
+		if err.Error() == "server should be stopped" {
 			return fmt.Errorf("server %s is already started: %v", server, err)
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -485,7 +485,7 @@ func StartServer(api *ScalewayAPI, needle string, wait bool) error {`
`485`	`485`
`486`	`486`	`err := api.PostServerAction(server, "poweron")`
`487`	`487`	`if err != nil {`
`488`		`- if err.Error() != "server should be stopped" {`
	`488`	`+ if err.Error() == "server should be stopped" {`
`489`	`489`	`return fmt.Errorf("server %s is already started: %v", server, err)`
`490`	`490`	`}`
`491`	`491`	`}`