Skip to content

Commit

Permalink
Fix helm charts
Browse files Browse the repository at this point in the history
  • Loading branch information
mihirgt committed Oct 10, 2024
1 parent 2454cd2 commit cf6889e
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 5 deletions.
1 change: 0 additions & 1 deletion helm/templates/config-service-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,4 +43,3 @@ data:
{{- end }}
]
}
12 changes: 8 additions & 4 deletions helm/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,15 +20,15 @@ spec:
metadata:
labels:
release: {{ .Release.Name }}
{{- with .Values.podLabels }}
{{- toYaml . | nindent 8 }}
{{- with merge .Values.podLabels .Values.commonPodLabels }}
{{- toYaml . | nindent 8 }}
{{- end }}
annotations:
checksum/config: {{ include (print $.Template.BasePath "/config-service-config.yaml") . | sha256sum }}
prometheus.io/scrape: "true"
prometheus.io/port: "{{ .Values.containerAdminPort }}"
{{- with .Values.podAnnotations }}
{{- toYaml . | nindent 8 }}
{{- with merge .Values.podAnnotations .Values.commonPodAnnotations }}
{{- toYaml . | nindent 8 }}
{{- end }}
spec:
{{- with .Values.imagePullSecrets }}
Expand Down Expand Up @@ -86,3 +86,7 @@ spec:
port: {{ .Values.containerAdminPort }}
resources:
{{- toYaml .Values.resources | nindent 12 }}
{{- with .Values.containerSecurityContext }}
securityContext:
{{- toYaml . | nindent 12 }}
{{- end }}
12 changes: 12 additions & 0 deletions helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,14 @@ service:
type: ClusterIP
port: 50101

containerSecurityContext: {}

podSecurityContext: {}

commonPodLabels: {}

commonPodAnnotations: {}

nodeLabels: {}

javaOpts: "-XX:InitialRAMPercentage=50.0 -XX:MaxRAMPercentage=75.0"
Expand Down Expand Up @@ -115,3 +123,7 @@ kafka-topic-creator:
address: zookeeper:2181
imagePullSecrets: []
podAnnotations: {}
podSecurityContext: {}
commonPodLabels: {}
containerSecurityContext: {}

0 comments on commit cf6889e

Please sign in to comment.