Merge pull request #8216 from sharifelgamal/volume-del

multinode: delete worker volumes for docker driver

Author: medyagh

Diff for: cmd/minikube/cmd/delete.go

@@ -271,7 +271,10 @@ func deleteProfile(profile *config.Profile) error {
 		// if driver is oci driver, delete containers and volumes
 		if driver.IsKIC(profile.Config.Driver) {
 			out.T(out.DeletingHost, `Deleting "{{.profile_name}}" in {{.driver_name}} ...`, out.V{"profile_name": profile.Name, "driver_name": profile.Config.Driver})
-			deletePossibleKicLeftOver(profile.Name, profile.Config.Driver)
+			for _, n := range profile.Config.Nodes {
+				machineName := driver.MachineName(*profile.Config, n)
+				deletePossibleKicLeftOver(machineName, profile.Config.Driver)
+			}
 		}
 	} else {
 		glog.Infof("%s has no configuration, will try to make it work anyways", profile.Name)

Diff for: cmd/minikube/cmd/start.go

@@ -316,7 +316,8 @@ func startWithDriver(starter node.Starter, existing *config.ClusterConfig) (*kub
 }
 
 func warnAboutMultiNode() {
-	out.WarningT("Multi-node clusters are currently experimental and might exhibit unintended behavior.\nTo track progress on multi-node clusters, see https://github.com/kubernetes/minikube/issues/7538.")
+	out.WarningT("Multi-node clusters are currently experimental and might exhibit unintended behavior.")
+	out.T(out.Documentation, "To track progress on multi-node clusters, see https://github.com/kubernetes/minikube/issues/7538.")
 }
 
 func updateDriver(driverName string) {

Diff for: pkg/minikube/machine/delete.go

@@ -99,7 +99,7 @@ func DeleteHost(api libmachine.API, machineName string, deleteAbandoned ...bool)
 	return delete(api, host, machineName)
 }
 
-// delete removes a host and it's local data files
+// delete removes a host and its local data files
 func delete(api libmachine.API, h *host.Host, machineName string) error {
 	if err := h.Driver.Remove(); err != nil {
 		glog.Warningf("remove failed, will retry: %v", err)