fix: add nodePort to service (#213)

Co-authored-by: Christian Hopf <git@ckotzbauer.de>

charts/cadvisor/templates/service.yaml

@@ -11,6 +11,9 @@ spec:
     port: {{ .Values.container.port }}
     targetPort: {{ .Values.container.port }}
     protocol: TCP
+    {{- with .Values.service.nodePort }}
+    nodePort: {{ . }}
+    {{- end }}
   selector:
     {{- include "cadvisor.selectorLabels" . | nindent 4 }}
-  type: ClusterIP
+  type: {{ .Values.service.type }}

charts/cadvisor/values.yaml

@@ -35,6 +35,10 @@ container:
       path: "/dev/disk"
       readOnly: true
 
+service:
+  type: ClusterIP
+  nodePort:  # If type is NodePort, specify the nodePort value. (30000-32767)
+
 resources: {}
   # We usually recommend not to specify default resources and to leave this as a conscious
   # choice for the user. This also increases chances charts run on environments with little