Skip to content

Commit

Permalink
[Helm][K8S] Add rootPath helm (#1593)
Browse files Browse the repository at this point in the history
* Update of values.yaml to use a new variable to manage the application rootpath

* Use rootPath of values in deployment to manage rootPath and probes
  • Loading branch information
jbwittner authored Jul 24, 2024
1 parent bfe38c7 commit e07f73d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
8 changes: 5 additions & 3 deletions chart/stirling-pdf/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,10 @@ spec:
imagePullPolicy: {{ .Values.image.pullPolicy }}
securityContext:
{{- toYaml .Values.containerSecurityContext | nindent 10 }}
{{- if .Values.envs }}
env:
- name: SYSTEM_ROOTURIPATH
value: {{ .Values.rootPath}}
{{- if .Values.envs }}
{{ toYaml .Values.envs | indent 8 }}
{{- end }}
{{- if .Values.extraArgs }}
Expand All @@ -75,13 +77,13 @@ spec:
containerPort: 8080
livenessProbe:
httpGet:
path: /
path: {{ .Values.rootPath}}
port: http
{{ toYaml .Values.probes.livenessHttpGetConfig | indent 12 }}
{{ toYaml .Values.probes.liveness | indent 10 }}
readinessProbe:
httpGet:
path: /
path: {{ .Values.rootPath}}
port: http
{{ toYaml .Values.probes.readinessHttpGetConfig | indent 12 }}
{{ toYaml .Values.probes.readiness | indent 10 }}
Expand Down
5 changes: 3 additions & 2 deletions chart/stirling-pdf/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ secret:
commonLabels: {}
# team_name: dev

# rootpath for the application
rootPath: /

envs: []
# - name: UI_APP_NAME
# value: "Stirling PDF"
Expand All @@ -24,8 +27,6 @@ envs: []
# value: "Stirling PDF"
# - name: ALLOW_GOOGLE_VISIBILITY
# value: "true"
# - name: APP_ROOT_PATH
# value: "/"
# - name: APP_LOCALE
# value: "en_GB"

Expand Down

1 comment on commit e07f73d

@Plataformaalgoritmica-svg

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello World

Please sign in to comment.