Do not require "parallels" driver binary since it's built-in to minikube

kubernetes · Oct 21, 2020 · bb25d9a · bb25d9a
1 parent 0758a7f
commit bb25d9a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 22 deletions.
diff --git a/pkg/minikube/registry/drvs/parallels/parallels.go b/pkg/minikube/registry/drvs/parallels/parallels.go
@@ -55,9 +55,9 @@ func configure(cfg config.ClusterConfig, n config.Node) (interface{}, error) {
 }
 
 func status() registry.State {
-	_, err := exec.LookPath("docker-machine-driver-parallels")
+	_, err := exec.LookPath("prlctl")
 	if err != nil {
-		return registry.State{Error: err, Fix: "Install docker-machine-driver-parallels", Doc: "https://minikube.sigs.k8s.io/docs/reference/drivers/parallels/"}
+		return registry.State{Error: err, Fix: "Install Parallels Desktop for Mac", Doc: "https://minikube.sigs.k8s.io/docs/drivers/parallels/"}
 	}
-	return registry.State{Installed: true, Healthy: true, Running: true}
+	return registry.State{Installed: true, Healthy: true}
 }
diff --git a/site/content/en/docs/drivers/includes/parallels_usage.inc b/site/content/en/docs/drivers/includes/parallels_usage.inc
@@ -2,24 +2,6 @@
 
 * Parallels Desktop 11.0.0+ Pro or Business edition
 
-## Driver Installation
-
-If the [Brew Package Manager](https://brew.sh/) is installed, run:
-
-```shell
-brew install docker-machine-parallels
-```
-
-Otherwise:
-
-```shell
-latest=$(curl -s https://api.github.com/repos/Parallels/docker-machine-parallels/releases/latest \
-  | grep -o 'http.*docker-machine-driver-parallels' | head -n1)
-curl -LO $latest
-install docker-machine-driver-parallels /usr/local/bin/
-```
-
-
 ## Usage
 
 Start a cluster using the parallels driver:

diff --git a/site/content/en/docs/drivers/parallels.md b/site/content/en/docs/drivers/parallels.md
@@ -7,7 +7,7 @@ aliases:
 
 ## Overview
 
-The Parallels driver is particularly useful for users who own Parallels Desktop, as it does not require VT-x hardware support.
+The Parallels driver is particularly useful for users who own Parallels Desktop for Mac, as it does not require VT-x hardware support.
 
 {{% readfile file="/docs/drivers/includes/parallels_usage.inc" %}}