store: add StatusSaved to the Instance Status

Signed-off-by: Norio Nomura <[email protected]>

Author: norio-nomura

cmd/limactl/copy.go

@@ -77,6 +77,8 @@ func copyAction(cmd *cobra.Command, args []string) error {
 			}
 			if inst.Status == store.StatusStopped {
 				return fmt.Errorf("instance %q is stopped, run `limactl start %s` to start the instance", instName, instName)
+			} else if inst.Status == store.StatusSaved {
+				return fmt.Errorf("instance %q is saved, run `limactl start %s` to restore the instance", instName, instName)
 			}
 			if legacySSH {
 				scpFlags = append(scpFlags, "-P", fmt.Sprintf("%d", inst.SSHLocalPort))

cmd/limactl/disk.go

@@ -331,6 +331,9 @@ func diskUnlockAction(_ *cobra.Command, args []string) error {
 			if inst.Status == store.StatusRunning {
 				logrus.Warnf("Cannot unlock disk %q used by running instance %q", diskName, disk.Instance)
 				continue
+			} else if inst.Status == store.StatusSaved {
+				logrus.Warnf("Cannot unlock disk %q used by saved instance %q", diskName, disk.Instance)
+				continue
 			}
 		}
 		if err := disk.Unlock(); err != nil {
@@ -387,6 +390,8 @@ func diskResizeAction(cmd *cobra.Command, args []string) error {
 		if err == nil {
 			if inst.Status == store.StatusRunning {
 				return fmt.Errorf("cannot resize disk %q used by running instance %q. Please stop the VM instance", diskName, disk.Instance)
+			} else if inst.Status == store.StatusSaved {
+				return fmt.Errorf("cannot resize disk %q used by saved instance %q. Please restore and stop the VM instance", diskName, disk.Instance)
 			}
 		}
 	}

cmd/limactl/edit.go

@@ -68,6 +68,8 @@ func editAction(cmd *cobra.Command, args []string) error {
 
 		if inst.Status == store.StatusRunning {
 			return errors.New("cannot edit a running instance")
+		} else if inst.Status == store.StatusSaved {
+			return errors.New("cannot edit a saved instance")
 		}
 		filePath = filepath.Join(inst.Dir, filenames.LimaYAML)
 	}

cmd/limactl/shell.go

@@ -77,6 +77,8 @@ func shellAction(cmd *cobra.Command, args []string) error {
 	}
 	if inst.Status == store.StatusStopped {
 		return fmt.Errorf("instance %q is stopped, run `limactl start %s` to start the instance", instName, instName)
+	} else if inst.Status == store.StatusSaved {
+		return fmt.Errorf("instance %q is saved, run `limactl start %s` to restore the instance", instName, instName)
 	}
 
 	// When workDir is explicitly set, the shell MUST have workDir as the cwd, or exit with an error.

cmd/limactl/start.go

@@ -490,7 +490,7 @@ func startAction(cmd *cobra.Command, args []string) error {
 			inst.Name, instance.LimactlShellCmd(inst.Name))
 		// Not an error
 		return nil
-	case store.StatusStopped:
+	case store.StatusStopped, store.StatusSaved:
 		// NOP
 	default:
 		logrus.Warnf("expected status %q, got %q", store.StatusStopped, inst.Status)

cmd/limactl/tunnel.go

@@ -70,6 +70,8 @@ func tunnelAction(cmd *cobra.Command, args []string) error {
 	}
 	if inst.Status == store.StatusStopped {
 		return fmt.Errorf("instance %q is stopped, run `limactl start %s` to start the instance", instName, instName)
+	} else if inst.Status == store.StatusSaved {
+		return fmt.Errorf("instance %q is saved, run `limactl start %s` to restore the instance", instName, instName)
 	}
 
 	if port == 0 {

pkg/instance/stop.go

@@ -17,6 +17,7 @@ import (
 )
 
 func StopGracefully(inst *store.Instance) error {
+	// TODO: support store.StatusSuspended
 	if inst.Status != store.StatusRunning {
 		return fmt.Errorf("expected status %q, got %q (maybe use `limactl stop -f`?)", store.StatusRunning, inst.Status)
 	}

pkg/store/instance.go

@@ -36,6 +36,7 @@ const (
 	StatusBroken        Status = "Broken"
 	StatusStopped       Status = "Stopped"
 	StatusRunning       Status = "Running"
+	StatusSaved         Status = "Saved"
 )
 
 type Instance struct {
@@ -198,7 +199,15 @@ func inspectStatusWithPIDFiles(instDir string, inst *Instance, y *limayaml.LimaY
 		case inst.HostAgentPID > 0 && inst.DriverPID > 0:
 			inst.Status = StatusRunning
 		case inst.HostAgentPID == 0 && inst.DriverPID == 0:
-			inst.Status = StatusStopped
+			_, err = os.Stat(filepath.Join(instDir, filenames.VzMachineState))
+			if err == nil {
+				inst.Status = StatusSaved
+			} else if errors.Is(err, os.ErrNotExist) {
+				inst.Status = StatusStopped
+			} else {
+				inst.Errors = append(inst.Errors, err)
+				inst.Status = StatusBroken
+			}
 		case inst.HostAgentPID > 0 && inst.DriverPID == 0:
 			inst.Errors = append(inst.Errors, errors.New("host agent is running but driver is not"))
 			inst.Status = StatusBroken