DevFW-CICD/argocd-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Disable deploying ui service when ui is disabled

Browse source 
**Why**
We had previoulsy removed the UI deployment, but not the service. This potentially poses a security hole.
This commit is contained in:
Lukasz Lempart 2018-09-24 19:43:26 -07:00
parent 7343110ebf
commit c3f6c2626d
2 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
charts/argo/Chart.yaml
View file

@ -1,4 +1,4 @@
apiVersion: v1 apiVersion: v1
description: A Helm chart for Kubernetes description: A Helm chart for Kubernetes
name: argo name: argo
version: 0.2.0 version: 0.2.1

2
charts/argo/templates/ui-service.yaml
View file

@ -1,3 +1,4 @@
{{ if .Values.ui.enabled }}
apiVersion: v1 apiVersion: v1
kind: Service kind: Service
metadata: metadata:
@ -15,3 +16,4 @@ spec:
app: {{ .Release.Name }}-{{ .Values.uiName}} app: {{ .Release.Name }}-{{ .Values.uiName}}
sessionAffinity: None sessionAffinity: None
type: {{ .Values.uiServiceType }} type: {{ .Values.uiServiceType }}
{{ end }}