hyperkit driver should be happy with current minimum verison

pkg/minikube/driver/install.go

@@ -55,7 +55,7 @@ func InstallOrUpdate(name string, directory string, v semver.Version, interactiv
 	defer releaser.Release()
 
 	exists := driverExists(executable)
-	path, err := validateDriver(executable, v)
+	path, err := validateDriver(executable, minDriverVersion(name, v))
 	if !exists || (err != nil && autoUpdate) {
 		glog.Warningf("%s: %v", executable, err)
 		path = filepath.Join(directory, executable)
@@ -133,6 +133,7 @@ func validateDriver(executable string, v semver.Version) (string, error) {
 	if err != nil {
 		return path, errors.Wrap(err, "can't parse driver version")
 	}
+
 	if driverVersion.LT(v) {
 		return path, fmt.Errorf("%s is version %s, want %s", executable, driverVersion, v)
 	}

pkg/minikube/driver/version.go

@@ -0,0 +1,31 @@
+package driver
+
+import (
+	"github.com/blang/semver"
+	"github.com/golang/glog"
+)
+
+// minHyperkitVersion is the minimum version of the minikube hyperkit driver compatible with the current minikube code
+var minHyperkitVersion semver.Version
+
+const minHyperkitVersionStr = "1.10.0"
+
+func init() {
+	v, err := semver.New(minHyperkitVersionStr)
+	if err != nil {
+		glog.Fatalf("Failed to parse the hyperkit driver version: %v", err)
+	}
+	minHyperkitVersion = *v
+}
+
+func minDriverVersion(driver string, mkVer semver.Version) semver.Version {
+	switch driver {
+	case HyperKit:
+		return minHyperkitVersion
+	case KVM2:
+		return mkVer
+	default:
+		glog.Warningf("Unexpected driver: %v", driver)
+		return mkVer
+	}
+}

pkg/minikube/driver/version_test.go

@@ -0,0 +1,36 @@
+package driver
+
+import (
+	"testing"
+
+	"github.com/blang/semver"
+)
+
+func Test_minDriverVersion(t *testing.T) {
+
+	tests := []struct {
+		desc   string
+		driver string
+		mkV    string
+		want   semver.Version
+	}{
+		{"Hyperkit", HyperKit, "1.1.1", minHyperkitVersion},
+		{"Invalid", "_invalid_", "1.1.1", v("1.1.1")},
+		{"KVM2", KVM2, "1.1.1", v("1.1.1")},
+	}
+	for _, tt := range tests {
+		t.Run(tt.desc, func(t *testing.T) {
+			if got := minDriverVersion(tt.driver, v(tt.mkV)); !got.EQ(tt.want) {
+				t.Errorf("minDriverVersion() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}
+
+func v(s string) semver.Version {
+	r, err := semver.New(s)
+	if err != nil {
+		panic(err)
+	}
+	return *r
+}