From 2a38fa3cfc5a534a459210c9a75c05db20f08114 Mon Sep 17 00:00:00 2001 From: Medya Gh Date: Thu, 16 Apr 2020 23:44:07 -0700 Subject: [PATCH 1/2] enable tunnel and service for docker on windows --- cmd/minikube/cmd/service.go | 4 ++-- cmd/minikube/cmd/tunnel.go | 5 +++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/cmd/minikube/cmd/service.go b/cmd/minikube/cmd/service.go index 039afefc28..6966f3209e 100644 --- a/cmd/minikube/cmd/service.go +++ b/cmd/minikube/cmd/service.go @@ -23,7 +23,6 @@ import ( "os" "os/signal" "path/filepath" - "runtime" "strconv" "strings" "text/template" @@ -34,6 +33,7 @@ import ( "k8s.io/minikube/pkg/drivers/kic/oci" "k8s.io/minikube/pkg/minikube/browser" + "k8s.io/minikube/pkg/minikube/driver" "k8s.io/minikube/pkg/minikube/exit" "k8s.io/minikube/pkg/minikube/localpath" "k8s.io/minikube/pkg/minikube/mustload" @@ -78,7 +78,7 @@ var serviceCmd = &cobra.Command{ cname := ClusterFlagValue() co := mustload.Healthy(cname) - if runtime.GOOS == "darwin" && co.Config.Driver == oci.Docker { + if driver.NeedsPortForward(co.Config.Driver) { startKicServiceTunnel(svc, cname) return } diff --git a/cmd/minikube/cmd/tunnel.go b/cmd/minikube/cmd/tunnel.go index dbf66cf110..9edb8086c5 100644 --- a/cmd/minikube/cmd/tunnel.go +++ b/cmd/minikube/cmd/tunnel.go @@ -21,7 +21,6 @@ import ( "os" "os/signal" "path/filepath" - "runtime" "strconv" "time" @@ -30,6 +29,7 @@ import ( "k8s.io/minikube/pkg/drivers/kic/oci" "k8s.io/minikube/pkg/minikube/config" + "k8s.io/minikube/pkg/minikube/driver" "k8s.io/minikube/pkg/minikube/exit" "k8s.io/minikube/pkg/minikube/localpath" "k8s.io/minikube/pkg/minikube/mustload" @@ -78,7 +78,8 @@ var tunnelCmd = &cobra.Command{ cancel() }() - if runtime.GOOS == "darwin" && co.Config.Driver == oci.Docker { + if driver.NeedsPortForward(co.Config.Driver) { + port, err := oci.ForwardedPort(oci.Docker, cname, 22) if err != nil { exit.WithError("error getting ssh port", err) From 27bd2a9e0b48a96f2123278d6601a00e1ee7a4ea Mon Sep 17 00:00:00 2001 From: Medya Gh Date: Fri, 17 Apr 2020 00:28:40 -0700 Subject: [PATCH 2/2] fix message --- cmd/minikube/cmd/service.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cmd/minikube/cmd/service.go b/cmd/minikube/cmd/service.go index 6966f3209e..f9e222d649 100644 --- a/cmd/minikube/cmd/service.go +++ b/cmd/minikube/cmd/service.go @@ -23,6 +23,7 @@ import ( "os" "os/signal" "path/filepath" + "runtime" "strconv" "strings" "text/template" @@ -137,7 +138,7 @@ func startKicServiceTunnel(svc, configName string) { service.PrintServiceList(os.Stdout, data) openURLs(svc, urls) - out.WarningT("Because you are using docker driver on Mac, the terminal needs to be open to run it.") + out.WarningT("Because you are using a Docker driver on {{.operating_system}}, the terminal needs to be open to run it.", out.V{"operating_system": runtime.GOOS}) <-ctrlC