Merge pull request #6315 from medyagh/istio_disable

disable istio provisioner by default

Author: sharifelgamal

deploy/addons/istio/README.md

@@ -1,11 +1,16 @@
 ## istio Addon
 [istio](https://istio.io/docs/setup/getting-started/) - Cloud platforms provide a wealth of benefits for the organizations that use them.
 
-### Enabling istio
-Propose to startup minikube with at least 8192 MB of memory and 4 CPUs to enable istio.
-To enable this addon, simply run:
+### Enable istio on minikube
+Make sure to start minikube with at least 8192 MB of memory and 4 CPUs.
+
+```shell script
+minikube start --memory=8000mb --cpus=4 
+```
 
+To enable this addon, simply run:
 ```shell script
+minikube addons enable istio-provisioner
 minikube addons enable istio
 ```
 
@@ -19,8 +24,9 @@ kubectl get po -n istio-system
 
 If everything went well you shouldn't get any errors about istio being installed in your cluster. If you haven't deployed any releases `kubectl get po -n istio-system` won't return anything.
 
-### Deprecation of istio
+### Disable istio
 To disable this addon, simply run:
 ```shell script
+minikube addons disable istio-provisioner
 minikube addons disable istio
-```
+```

pkg/minikube/assets/addons.go

@@ -195,7 +195,7 @@ var Addons = map[string]*Addon{
 			"istio-operator.yaml",
 			"0640",
 			true),
-	}, true, "istio-provisioner"),
+	}, false, "istio-provisioner"),
 	"istio": NewAddon([]*BinAsset{
 		MustBinAsset(
 			"deploy/addons/istio/istio-default-profile.yaml.tmpl",