fix logging non-existent path

Signed-off-by: Songpon Srisawai <songpon.ssw@gmail.com>

Author: songponssw

pkg/limayaml/validate.go

@@ -144,42 +144,48 @@ func Validate(y *LimaYAML, warn bool) error {
 		}
 	}
 
+	var mountErrs []error
 	for i, f := range y.Mounts {
 		if !filepath.IsAbs(f.Location) && !strings.HasPrefix(f.Location, "~") {
-			return fmt.Errorf("field `mounts[%d].location` must be an absolute path, got %q",
-				i, f.Location)
+			mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].location` must be an absolute path, got %q", i, f.Location))
 		}
 		// f.Location has already been expanded in FillDefaults(), but that function cannot return errors.
 		loc, err := localpathutil.Expand(f.Location)
 		if err != nil {
-			return fmt.Errorf("field `mounts[%d].location` refers to an unexpandable path: %q: %w", i, f.Location, err)
+			mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].location` refers to an unexpandable path: %q: %w", i, f.Location, err))
 		}
 		st, err := os.Stat(loc)
 		if err != nil {
-			if !errors.Is(err, os.ErrNotExist) {
-				return fmt.Errorf("field `mounts[%d].location` refers to an inaccessible path: %q: %w", i, f.Location, err)
+			if errors.Is(err, os.ErrNotExist) {
+				mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].location` refers to non-existent path: %q: %w", i, f.Location, err))
+				continue
 			}
+			mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].location` refers to an inaccessible path: %q: %w", i, f.Location, err))
 		} else if !st.IsDir() {
-			return fmt.Errorf("field `mounts[%d].location` refers to a non-directory path: %q: %w", i, f.Location, err)
+			mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].location` refers to a non-directory path: %q: %w", i, f.Location, err))
 		}
 
 		switch *f.MountPoint {
 		case "/", "/bin", "/dev", "/etc", "/home", "/opt", "/sbin", "/tmp", "/usr", "/var":
-			return fmt.Errorf("field `mounts[%d].mountPoint` must not be a system path such as /etc or /usr", i)
+			mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].mountPoint` must not be a system path such as /etc or /usr", i))
 		// home directory defined in "cidata.iso:/user-data"
 		case *y.User.Home:
-			return fmt.Errorf("field `mounts[%d].mountPoint` is the reserved internal home directory %q", i, *y.User.Home)
+			mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].mountPoint` is the reserved internal home directory %q", i, *y.User.Home))
 		}
 		// There is no tilde-expansion for guest filenames
 		if strings.HasPrefix(*f.MountPoint, "~") {
-			return fmt.Errorf("field `mounts[%d].mountPoint` must not start with \"~\"", i)
+			mountErrs = append(mountErrs, fmt.Errorf("field `mounts[%d].mountPoint` must not start with \"~\"", i))
 		}
 
 		if _, err := units.RAMInBytes(*f.NineP.Msize); err != nil {
-			return fmt.Errorf("field `msize` has an invalid value: %w", err)
+			mountErrs = append(mountErrs, fmt.Errorf("field `msize` has an invalid value: %w", err))
 		}
 	}
 
+	if len(mountErrs) > 0 {
+		return fmt.Errorf("mounts validation failed:\n%w", errors.Join(mountErrs...))
+	}
+
 	if *y.SSH.LocalPort != 0 {
 		if err := validatePort("ssh.localPort", *y.SSH.LocalPort); err != nil {
 			return err