Skip to content

Commit 37adeeb

Browse files
authored
Some minor chart improvements (#187)
Signed-off-by: Sergio Castaño Arteaga <[email protected]>
1 parent f7d7653 commit 37adeeb

File tree

4 files changed

+19
-8
lines changed

4 files changed

+19
-8
lines changed

charts/ocg/templates/server_deployment.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ spec:
2828
- {{- include "chart.checkDbIsReadyInitContainer" . | nindent 10 }}
2929
{{- if .Release.IsInstall }}
3030
- name: check-dbmigrator-run
31-
image: "bitnami/kubectl:{{ template "chart.KubernetesVersion" . }}"
31+
{{ $kubeVersion := include "chart.KubernetesVersion" . }}
32+
{{ $kubectlImageVersion := ternary "1.33" $kubeVersion (semverCompare ">=1.34.0-0" (printf "%s.0" $kubeVersion)) }}
33+
image: "docker.io/bitnamilegacy/kubectl:{{ $kubectlImageVersion }}"
3234
imagePullPolicy: IfNotPresent
3335
command: ['kubectl', 'wait', '--namespace={{ .Release.Namespace }}', '--for=condition=complete', 'job/{{ include "chart.resourceNamePrefix" . }}dbmigrator-install', '--timeout=60s']
3436
{{- end }}

charts/ocg/templates/server_ingress.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ spec:
1515
service:
1616
name: {{ include "chart.resourceNamePrefix" . }}server
1717
port:
18-
number: {{ .Values.server.service.port }}
18+
number: {{ .Values.server.ingress.backendServicePort }}
1919
{{- with .Values.server.ingress.rules }}
2020
rules:
2121
{{- toYaml . | nindent 4 }}

charts/ocg/templates/server_service.yaml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,15 @@ metadata:
55
labels:
66
app.kubernetes.io/component: server
77
{{- include "chart.labels" . | nindent 4 }}
8+
{{- with .Values.server.service.annotations }}
9+
annotations:
10+
{{- toYaml . | nindent 4 }}
11+
{{- end }}
812
spec:
13+
allocateLoadBalancerNodePorts: {{ .Values.server.service.allocateLoadBalancerNodePorts }}
914
type: {{ .Values.server.service.type }}
1015
ports:
11-
- port: {{ .Values.server.service.port }}
12-
targetPort: http
13-
protocol: TCP
14-
name: http
16+
{{- toYaml .Values.server.service.ports | nindent 4 }}
1517
selector:
1618
app.kubernetes.io/component: server
1719
{{- include "chart.selectorLabels" . | nindent 4 }}

charts/ocg/values.yaml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,13 +120,20 @@ server:
120120
enabled: true
121121
annotations:
122122
kubernetes.io/ingress.class: nginx
123+
backendServicePort: 80
123124
rules: []
124125
tls: []
125126

126127
# Service configuration
127128
service:
129+
allocateLoadBalancerNodePorts: true
130+
annotations: {}
131+
ports:
132+
- name: http
133+
port: 80
134+
protocol: TCP
135+
targetPort: 9000
128136
type: NodePort
129-
port: 80
130137

131138
# Deployment configuration
132139
deploy:
@@ -144,7 +151,7 @@ postgresql:
144151
password: postgres
145152
database: ocg
146153
image:
147-
repository: bitnami/postgresql
154+
repository: docker.io/bitnamilegacy/postgresql
148155
tag: 17.2.0-debian-12-r3
149156
postgresqlDataDir: /data/pgdata
150157
primary:

0 commit comments

Comments
 (0)