helm: Update hubble-ui chart

- Use v0.5.0 tag by default instead of latest. - Rely on requirements.yaml to enable the chart instead of guarding individual templates. Ref: https://github.com/cilium/hubble-ui/releases Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
cilium · Apr 30, 2020 · 9a9bab7 · 9a9bab7
1 parent 4b865d5
commit 9a9bab7
Show file tree

Hide file tree

Showing 7 changed files with 5 additions and 13 deletions.
diff --git a/install/kubernetes/cilium/charts/hubble-ui/templates/clusterrole.yaml b/install/kubernetes/cilium/charts/hubble-ui/templates/clusterrole.yaml
@@ -1,4 +1,3 @@
-{{- if .Values.global.hubble.ui.enabled }}
 kind: ClusterRole
 apiVersion: rbac.authorization.k8s.io/v1
 metadata:
@@ -41,4 +40,3 @@ rules:
       - get
       - list
       - watch
-{{- end }}
diff --git a/install/kubernetes/cilium/charts/hubble-ui/templates/clusterrolebinding.yaml b/install/kubernetes/cilium/charts/hubble-ui/templates/clusterrolebinding.yaml
@@ -1,4 +1,3 @@
-{{- if .Values.global.hubble.ui.enabled }}
 kind: ClusterRoleBinding
 apiVersion: rbac.authorization.k8s.io/v1
 metadata:
@@ -11,4 +10,3 @@ subjects:
   - kind: ServiceAccount
     namespace: {{ .Release.Namespace }}
     name: hubble-ui 
-{{- end }}
diff --git a/install/kubernetes/cilium/charts/hubble-ui/templates/deployment.yaml b/install/kubernetes/cilium/charts/hubble-ui/templates/deployment.yaml
@@ -1,4 +1,3 @@
-{{- if .Values.global.hubble.ui.enabled }}
 kind: Deployment
 apiVersion: apps/v1
 metadata:
@@ -35,4 +34,3 @@ spec:
               name: http
           resources:
             {{- toYaml .Values.resources | trim | nindent 12 }}
-{{- end }}
diff --git a/install/kubernetes/cilium/charts/hubble-ui/templates/ingress.yaml b/install/kubernetes/cilium/charts/hubble-ui/templates/ingress.yaml
@@ -1,4 +1,4 @@
-{{- if and .Values.global.hubble.ui.enabled .Values.ingress.enabled -}}
+{{- if .Values.ingress.enabled -}}
 {{- $ingressPath := .Values.ingress.path -}}
 apiVersion: extensions/v1beta1
 kind: Ingress

diff --git a/install/kubernetes/cilium/charts/hubble-ui/templates/serviceaccount.yaml b/install/kubernetes/cilium/charts/hubble-ui/templates/serviceaccount.yaml
@@ -1,7 +1,5 @@
-{{- if .Values.global.hubble.ui.enabled }}
 apiVersion: v1
 kind: ServiceAccount
 metadata:
   namespace: {{ .Release.Namespace }}
   name: hubble-ui 
-{{- end }}
diff --git a/install/kubernetes/cilium/charts/hubble-ui/templates/svc.yaml b/install/kubernetes/cilium/charts/hubble-ui/templates/svc.yaml
@@ -1,4 +1,3 @@
-{{- if .Values.global.hubble.ui.enabled }}
 kind: Service
 apiVersion: v1
 metadata:
@@ -12,4 +11,3 @@ spec:
       port: 12000
       targetPort: 12000
   type: ClusterIP
-{{- end }}
diff --git a/install/kubernetes/cilium/charts/hubble-ui/values.yaml b/install/kubernetes/cilium/charts/hubble-ui/values.yaml
@@ -2,8 +2,10 @@
 image:
   # repository of the docker image
   repository: quay.io/cilium/hubble-ui
-  # tag is the container image tag to use
-  tag: latest
+  # tag is the container image tag to use.
+  # Ref: https://github.com/cilium/hubble-ui/releases
+  # Ref: https://quay.io/repository/cilium/hubble-ui?tab=tags
+  tag: v0.5.0
   # pullPolicy is the container image pull policy
   pullPolicy: Always
 clusterDomain: cluster.local