Merge pull request #1575 from AkihiroSuda/vz-faily-early-invalid-arch

vz: fail early when arch is invalid

Author: AkihiroSuda

pkg/limayaml/validate.go

@@ -48,6 +48,17 @@ func Validate(y LimaYAML, warn bool) error {
 		return fmt.Errorf("field `arch` must be %q, %q, or %q; got %q", X8664, AARCH64, RISCV64, *y.Arch)
 	}
 
+	switch *y.VMType {
+	case QEMU:
+		// NOP
+	case VZ:
+		if !IsNativeArch(*y.Arch) {
+			return fmt.Errorf("field `arch` must be %q for VZ; got %q", NewArch(runtime.GOARCH), *y.Arch)
+		}
+	default:
+		return fmt.Errorf("field `vmType` must be %q or %q; got %q", QEMU, VZ, *y.VMType)
+	}
+
 	if len(y.Images) == 0 {
 		return errors.New("field `images` must be set")
 	}

pkg/vz/vz_driver_darwin.go

@@ -76,6 +76,10 @@ func (l *LimaVzDriver) Validate() error {
 		logrus.Warnf("Ignoring: vmType %s: %+v", *l.Yaml.VMType, unknown)
 	}
 
+	if !limayaml.IsNativeArch(*l.Yaml.Arch) {
+		return fmt.Errorf("unsupported arch: %q", *l.Yaml.Arch)
+	}
+
 	for k, v := range l.Yaml.CPUType {
 		if v != "" {
 			logrus.Warnf("Ignoring: vmType %s: cpuType[%q]: %q", *l.Yaml.VMType, k, v)