Skip to content

Commit

Permalink
Merge pull request #3564 from infrahq/mxyng/http-connector-helm
Browse files Browse the repository at this point in the history
fix(helm): http connector service
  • Loading branch information
mxyng authored Nov 10, 2022
2 parents 9ecce55 + ff47a5b commit 19ada18
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 0 deletions.
3 changes: 3 additions & 0 deletions helm/charts/infra/templates/connector/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,9 @@ spec:
{{- toYaml .Values.connector.volumeMounts | nindent 12 }}
{{- end }}
ports:
- name: http
containerPort: {{ .Values.connector.config.addr.http }}
protocol: TCP
- name: https
containerPort: {{ .Values.connector.config.addr.https }}
protocol: TCP
Expand Down
7 changes: 7 additions & 0 deletions helm/charts/infra/templates/connector/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,13 @@ spec:
sessionAffinity: {{ .Values.connector.service.sessionAffinity }}
{{- end }}
ports:
- port: {{ .Values.connector.service.port }}
name: {{ .Values.connector.service.portName }}
targetPort: http
protocol: TCP
{{- if eq .Values.connector.service.type "NodePort" }}
nodePort: {{ .Values.connector.service.nodePort }}
{{- end }}
- port: {{ .Values.connector.service.securePort }}
name: {{ .Values.connector.service.securePortName }}
targetPort: https
Expand Down
1 change: 1 addition & 0 deletions helm/charts/infra/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1061,6 +1061,7 @@ connector:

## Connector container service ports
addr:
http: 9080
https: 9443
metrics: 9091

Expand Down

0 comments on commit 19ada18

Please sign in to comment.