Merge pull request #49 from AkihiroSuda/dev

allow `limactl start <ALREADY_RUNNING_INSTANCE>` without raising an error

Author: AkihiroSuda

cmd/limactl/start.go

@@ -174,6 +174,17 @@ func startAction(clicontext *cli.Context) error {
 	if err != nil {
 		return err
 	}
+	switch inst.Status {
+	case store.StatusRunning:
+		logrus.Infof("The instance %q is already running. Run `%s` to open the shell.",
+			inst.Name, start.LimactlShellCmd(inst.Name))
+		// Not an error
+		return nil
+	case store.StatusStopped:
+		// NOP
+	default:
+		logrus.Warnf("expected status %q, got %q", store.StatusStopped, inst.Status)
+	}
 	ctx := clicontext.Context
 	return start.Start(ctx, inst)
 }

pkg/start/start.go

@@ -135,11 +135,7 @@ func watchHostAgentEvents(ctx context.Context, instName, haStdoutPath, haStderrP
 				return true
 			}
 
-			shellCmd := fmt.Sprintf("limactl shell %s", instName)
-			if instName == "default" {
-				shellCmd = "lima"
-			}
-			logrus.Infof("READY. Run `%s` to open the shell.", shellCmd)
+			logrus.Infof("READY. Run `%s` to open the shell.", LimactlShellCmd(instName))
 			err = nil
 			return true
 		}
@@ -160,3 +156,11 @@ func watchHostAgentEvents(ctx context.Context, instName, haStdoutPath, haStderrP
 
 	return nil
 }
+
+func LimactlShellCmd(instName string) string {
+	shellCmd := fmt.Sprintf("limactl shell %s", instName)
+	if instName == "default" {
+		shellCmd = "lima"
+	}
+	return shellCmd
+}