Skip to content

Commit

Permalink
Helm Chart: use port 8080 everywhere (#445)
Browse files Browse the repository at this point in the history
  • Loading branch information
tnotheis authored Dec 13, 2023
1 parent a02dc60 commit 2e2f137
Show file tree
Hide file tree
Showing 8 changed files with 13 additions and 15 deletions.
6 changes: 3 additions & 3 deletions helm/charts/adminui/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,20 +57,20 @@ spec:
imagePullPolicy: {{ .Values.image.pullPolicy }}
ports:
- name: http
containerPort: 80
containerPort: 8080
protocol: TCP
livenessProbe:
httpGet:
path: /
port: 80
port: 8080
initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }}
periodSeconds: {{ .Values.livenessProbe.periodSeconds }}
timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }}
failureThreshold: {{ .Values.livenessProbe.failureThreshold }}
readinessProbe:
httpGet:
path: /
port: 80
port: 8080
initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds }}
periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
timeoutSeconds: {{ .Values.readinessProbe.timeoutSeconds }}
Expand Down
4 changes: 2 additions & 2 deletions helm/charts/adminui/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ metadata:
spec:
type: {{ .Values.service.type }}
ports:
- port: 80
targetPort: {{ .Values.service.port }}
- port: 8080
targetPort: 8080
selector:
app: {{ include "global.name" . }}
2 changes: 1 addition & 1 deletion helm/charts/consumerapi/templates/backendconfig.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@ spec:
unhealthyThreshold: {{ .Values.backendConfig.healthCheck.unhealthyThreshold }}
type: HTTP
requestPath: /health
port: 80
port: 8080
{{- end }}
6 changes: 3 additions & 3 deletions helm/charts/consumerapi/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ spec:
imagePullPolicy: {{ .Values.image.pullPolicy }}
ports:
- name: http
containerPort: 80
containerPort: 8080
protocol: TCP
volumeMounts:
- name: settings-override
Expand All @@ -84,15 +84,15 @@ spec:
livenessProbe:
httpGet:
path: /health
port: 80
port: 8080
initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }}
periodSeconds: {{ .Values.livenessProbe.periodSeconds }}
timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }}
failureThreshold: {{ .Values.livenessProbe.failureThreshold }}
readinessProbe:
httpGet:
path: /health
port: 80
port: 8080
initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds }}
periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
timeoutSeconds: {{ .Values.readinessProbe.timeoutSeconds }}
Expand Down
2 changes: 1 addition & 1 deletion helm/charts/consumerapi/templates/httproute.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,5 @@ spec:
value: /
backendRefs:
- name: {{ include "global.name" . }}
port: 80
port: 8080
{{- end }}
2 changes: 1 addition & 1 deletion helm/charts/consumerapi/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,5 @@ spec:
service:
name: {{ include "global.name" . }}
port:
number: 80
number: 8080
{{- end }}
4 changes: 2 additions & 2 deletions helm/charts/consumerapi/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ spec:
loadBalancerIP: {{ .Values.service.loadBalancer.ip }}
{{- end }}
ports:
- port: 80
targetPort: {{ .Values.service.port }}
- port: 8080
targetPort: 8080
selector:
app: {{ include "global.name" . }}
2 changes: 0 additions & 2 deletions helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ consumerapi:

service:
type: "ClusterIP"
port: 8080
loadBalancer:
# ip - the static ip address the LoadBalancer should use
ip: ""
Expand Down Expand Up @@ -119,7 +118,6 @@ adminui:

service:
type: "ClusterIP"
port: 8080

image:
repository: "ghcr.io/nmshd/backbone-admin-ui"
Expand Down

0 comments on commit 2e2f137

Please sign in to comment.