From 1663a030ed55edc8fdcb68b25e4859c4c24b298a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Anders=20F=20Bj=C3=B6rklund?= <anders.f.bjorklund@gmail.com>
Date: Mon, 11 Jan 2021 18:12:06 +0100
Subject: [PATCH] Only add docker group for the docker runtime

---
 pkg/drivers/ssh/ssh.go        | 7 +++++++
 pkg/minikube/machine/start.go | 6 ------
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/pkg/drivers/ssh/ssh.go b/pkg/drivers/ssh/ssh.go
index b3228fc17..59cb8170f 100644
--- a/pkg/drivers/ssh/ssh.go
+++ b/pkg/drivers/ssh/ssh.go
@@ -20,6 +20,7 @@ import (
 	"fmt"
 	"net"
 	"os"
+	"os/exec"
 	"path"
 	"strconv"
 	"time"
@@ -124,6 +125,12 @@ func (d *Driver) Create() error {
 		}
 	}
 
+	if d.runtime.Name() == "Docker" {
+		if _, err := d.exec.RunCmd(exec.Command("sudo", "usermod", "-aG", "docker", d.GetSSHUsername())); err != nil {
+			return errors.Wrap(err, "usermod")
+		}
+	}
+
 	log.Debugf("IP: %s", d.IPAddress)
 
 	return nil
diff --git a/pkg/minikube/machine/start.go b/pkg/minikube/machine/start.go
index 7dd59164c..c664d69fd 100644
--- a/pkg/minikube/machine/start.go
+++ b/pkg/minikube/machine/start.go
@@ -174,12 +174,6 @@ func createHost(api libmachine.API, cfg *config.ClusterConfig, n *config.Node) (
 		return h, errors.Wrap(err, "post-start")
 	}
 
-	if driver.IsSSH(h.Driver.DriverName()) {
-		if _, err := h.RunSSHCommand(fmt.Sprintf("sudo usermod -aG docker %s", h.Driver.GetSSHUsername())); err != nil {
-			return h, errors.Wrap(err, "usermod")
-		}
-	}
-
 	if err := saveHost(api, h, cfg, n); err != nil {
 		return h, err
 	}
-- 
GitLab