add clarification comments on some implementation details, link documentation

Author: volatilemolotov

pkg/addons/helm.go

@@ -25,7 +25,7 @@ import (
 	"k8s.io/minikube/pkg/minikube/vmpath"
 )
 
-// runs a helm install based on the contents of chart *assets.HelmChart
+// runs a helm install within the minikube vm or container based on the contents of chart *assets.HelmChart
 func installHelmChart (ctx context.Context, chart *assets.HelmChart) *exec.Cmd {
 	args := []string{
 		fmt.Sprintf("KUBECONFIG=%s", path.Join(vmpath.GuestPersistentDir, "kubeconfig")),
@@ -80,6 +80,7 @@ func helmInstallBinary(addon *assets.Addon, runner command.Runner) error {
 
 		installCmd := fmt.Sprint("curl -fsSL -o get_helm.sh https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3 && chmod 700 get_helm.sh && ./get_helm.sh")
 		_, err = runner.RunCmd(exec.Command("sudo", "bash", "-c", installCmd))
+		// we copy the binary from /usr/local/bin to /usr/bin because /usr/local/bin is not in PATH in both iso and kicbase
 		_, err = runner.RunCmd(exec.Command("sudo", "mv", "/usr/local/bin/helm", "/usr/bin/helm"))
 		if err != nil {
 		return errors.Wrap(err, "installing helm")

site/content/en/docs/contrib/addons.en.md

@@ -93,6 +93,8 @@ The boolean value on the last line is whether the addon should be enabled by def
 
 To see other examples, see the [addons commit history](https://github.com/kubernetes/minikube/commits/master/deploy/addons) for other recent examples.
 
+To add a helm based addon please check out [Helm Based Addons](/docs/contrib/addons)
+
 ## "addons open" support
 
 If your addon contains a NodePort Service, please add the `kubernetes.io/minikube-addons-endpoint: <addon name>` label, which is used by the  `minikube addons open` command: