Add Helm documentation (#517)

* Add required blocks to Helm templates

Signed-off-by: lucperkins <lucperkins@gmail.com>

* Make CLI table border lighter

Signed-off-by: lucperkins <lucperkins@gmail.com>

* Change aesthetic of CLI table

Signed-off-by: lucperkins <lucperkins@gmail.com>

* Add admonition blocks

Signed-off-by: lucperkins <lucperkins@gmail.com>

* Fix wording

Signed-off-by: lucperkins <lucperkins@gmail.com>

charts/virtual-kubelet/templates/deployment.yaml

@@ -45,7 +45,7 @@ spec:
           value: {{ tpl .Values.taint.value $ }}
         - name: VKUBELET_TAINT_EFFECT
           value: {{ .Values.taint.effect }}
-{{- if eq .Values.provider "azure" }}
+{{- if eq (required "You must specify a Virtual Kubelet provider" .Values.provider) "azure" }}
 {{- with .Values.providers.azure }}
 {{- if .loganalytics.enabled }}
         - name: LOG_ANALYTICS_AUTH_LOCATION
@@ -113,7 +113,7 @@ spec:
         volumeMounts:
         - name: credentials
           mountPath: "/etc/virtual-kubelet"
-{{- if eq .Values.provider "azure" }}
+{{- if eq (required "You must specify a Virtual Kubelet provider" .Values.provider) "azure" }}
 {{- if .Values.providers.azure.targetAKS }}
         - name: acs-credential
           mountPath: "/etc/acs/azure.json"
@@ -124,7 +124,7 @@ spec:
 {{- if not .Values.taint.enabled }}
           "--disable-taint", "true",
 {{- end }}
-          "--provider", "{{ required "provider is required" .Values.provider }}",
+          "--provider", "{{ required "You must specify a Virtual Kubelet provider" .Values.provider }}",
           "--namespace", "{{ .Values.monitoredNamespace }}",
           "--nodename", "{{ required "nodeName is required" .Values.nodeName }}",
           {{- if .Values.logLevel }}
@@ -146,7 +146,7 @@ spec:
       - name: credentials
         secret:
           secretName: {{ template "vk.fullname" . }}
-{{- if eq .Values.provider "azure" }}
+{{- if eq (required "You must specify a Virtual Kubelet provider" .Values.provider) "azure" }}
 {{- if .Values.providers.azure.targetAKS }}
       - name: acs-credential
         hostPath:

charts/virtual-kubelet/templates/secret.yaml

@@ -17,7 +17,7 @@ data:
   cert.pem: {{ quote .Values.apiserverCert }}
   key.pem: {{ quote .Values.apiserverKey }}
 {{- end }}
-{{- if eq .Values.provider "azure" }}
+{{- if eq (required "You must specify a Virtual Kubelet provider" .Values.provider) "azure" }}
 {{- with .Values.providers.azure }}
 {{- if .loganalytics.enabled }}
   loganalytics.json: {{ printf "{\"workspaceID\": \"%s\",\"workspaceKey\": \"%s\"}" (required "workspaceId is required for loganalytics" .loganalytics.workspaceId ) (required "workspaceKey is required for loganalytics" .loganalytics.workspaceKey ) | b64enc | quote }}

charts/virtual-kubelet/values.yaml

@@ -3,8 +3,6 @@ image:
   tag: latest
   pullPolicy: Always
 
-## `provider` should be one of aws, azure, azurebatch, etc...
-provider: 
 nodeName: "virtual-kubelet"
 nodeOsType: "Linux"
 monitoredNamespace: ""