Skip to content

Commit

Permalink
[bitnami/redmine] Use different liveness/readiness probes (bitnami#25955
Browse files Browse the repository at this point in the history
)

Signed-off-by: David Gomez <[email protected]>
Signed-off-by: Fran Mulero <[email protected]>
  • Loading branch information
dgomezleon authored and fmulero committed May 22, 2024
1 parent fcaf76a commit 85960fa
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 11 deletions.
8 changes: 4 additions & 4 deletions bitnami/redmine/Chart.lock
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
dependencies:
- name: postgresql
repository: oci://registry-1.docker.io/bitnamicharts
version: 15.3.1
version: 15.3.2
- name: mariadb
repository: oci://registry-1.docker.io/bitnamicharts
version: 18.0.3
version: 18.0.5
- name: common
repository: oci://registry-1.docker.io/bitnamicharts
version: 2.19.2
digest: sha256:7fb4e0fbd51196a0bb3692bbff99ca092e6008f95da2a3c7391cd0f63f9113d5
generated: "2024-05-14T05:14:44.654300475Z"
digest: sha256:74033f13548423b7434216083ee6c5d769a432c2d4d074a9fb4a4e7cb33af1f0
generated: "2024-05-16T17:53:27.225232+02:00"
2 changes: 1 addition & 1 deletion bitnami/redmine/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,4 +43,4 @@ maintainers:
name: redmine
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/redmine
version: 28.0.3
version: 28.0.4
3 changes: 1 addition & 2 deletions bitnami/redmine/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ extraVolumeMounts:
## Configure extra options for liveness and readiness probes
## ref: https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-probes/#configure-probes)
##
livenessProbe:
startupProbe:
enabled: true
path: /redmine/
---
Expand Down Expand Up @@ -310,7 +310,6 @@ helm install test --set persistence.existingClaim=PVC_REDMINE,mariadb.persistenc
| `containerSecurityContext.capabilities.add` | List of capabilities to be added | `["CHOWN","SYS_CHROOT","FOWNER","SETGID","SETUID","DAC_OVERRIDE"]` |
| `containerSecurityContext.seccompProfile.type` | Set container's Security Context seccomp profile | `RuntimeDefault` |
| `livenessProbe.enabled` | Enable livenessProbe on Redmine containers | `true` |
| `livenessProbe.path` | Path for to check for livenessProbe | `/` |
| `livenessProbe.initialDelaySeconds` | Initial delay seconds for livenessProbe | `300` |
| `livenessProbe.periodSeconds` | Period seconds for livenessProbe | `10` |
| `livenessProbe.timeoutSeconds` | Timeout seconds for livenessProbe | `5` |
Expand Down
3 changes: 1 addition & 2 deletions bitnami/redmine/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -191,8 +191,7 @@ spec:
livenessProbe: {{- include "common.tplvalues.render" (dict "value" .Values.customLivenessProbe "context" $) | nindent 12 }}
{{- else if .Values.livenessProbe.enabled }}
livenessProbe: {{- include "common.tplvalues.render" (dict "value" (omit .Values.livenessProbe "enabled" "path") "context" $) | nindent 12 }}
httpGet:
path: {{ .Values.livenessProbe.path }}
tcpSocket:
port: http
{{- end }}
{{- if .Values.customReadinessProbe }}
Expand Down
2 changes: 0 additions & 2 deletions bitnami/redmine/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,6 @@ containerSecurityContext:
## Configure extra options for Redmine containers' liveness, readiness and startup probes
## ref: https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/#configure-probes
## @param livenessProbe.enabled Enable livenessProbe on Redmine containers
## @param livenessProbe.path Path for to check for livenessProbe
## @param livenessProbe.initialDelaySeconds Initial delay seconds for livenessProbe
## @param livenessProbe.periodSeconds Period seconds for livenessProbe
## @param livenessProbe.timeoutSeconds Timeout seconds for livenessProbe
Expand All @@ -252,7 +251,6 @@ containerSecurityContext:
##
livenessProbe:
enabled: true
path: /
initialDelaySeconds: 300
periodSeconds: 10
timeoutSeconds: 5
Expand Down

0 comments on commit 85960fa

Please sign in to comment.