zuul/zuul-helm
Code Issues Proposed changes

Allow override of service type for zuul-web

Browse Source 
To attach a service to a GCE loadbalancer, the service must be a
NodePort not a ClusterIP.  Allow users to specify that as an
option.  ClusterIP is the default.

Change-Id: I98a31f14c110293ac66d3a356accdf1e925663e5
This commit is contained in:
James E. Blair 2020-01-11 10:11:36 -08:00
parent 903a179745
commit 92886e64ca
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
charts/zuul/templates/web/service.yaml
View File

@ -8,7 +8,7 @@ metadata:
labels: labels:
{{- include "zuul.web.labels" . | indent 4 }} {{- include "zuul.web.labels" . | indent 4 }}
spec: spec:
type: ClusterIP type: {{ .Values.web.serviceType }}
ports: ports:
- name: zuul-web - name: zuul-web
port: {{ .Values.web.port }} port: {{ .Values.web.port }}

1
charts/zuul/values.yaml
View File

@ -20,6 +20,7 @@ web:
enabled: true enabled: true
replicas: 3 replicas: 3
port: 9000 port: 9000
serviceType: ClusterIP
ingress: ingress:
enabled: false enabled: false