Merge pull request #262 from QuentinPerez/fix_#171

Fix #171

Author: moul

README.md

@@ -1154,6 +1154,7 @@ $ scw inspect myserver | jq '.[0].public_ip.address'
 
 ### master (unreleased)
 
+# Add a warn message when using `ssh exec` on host without public ip nor gateway ([#171](https://github.com/scaleway/scaleway-cli/issues/171))
 * Support of `scw rmi` snapshot|volume ([#258](https://github.com/scaleway/scaleway-cli/issues/258))
 * Match bootscript/image with the good architecture ([#255](https://github.com/scaleway/scaleway-cli/issues/255))
 * Support of region/owner/arch in the cache file ([#255](https://github.com/scaleway/scaleway-cli/issues/255))

pkg/commands/exec.go

@@ -66,6 +66,10 @@ func RunExec(ctx CommandContext, args ExecArgs) error {
 		}
 	}
 
+	if server.PublicAddress.IP == "" && gateway == "" {
+		log.Warn(`Your host has no public IP address, you should use '--gateway', see 'scw help exec'`)
+	}
+
 	// --timeout
 	if args.Timeout > 0 {
 		log.Debugf("Setting up a global timeout of %d seconds", args.Timeout)