🐛(helm) reverse liveness and readiness for backend deployment

The liveness and readiness are reversed. The liveness was using the
heartbeat process that is cheking all django checks and the database
connection.
This commit is contained in:
Manuel Raynaud
2026-02-13 09:51:41 +01:00
parent 2cc0d71b89
commit 49871c45b1
2 changed files with 40 additions and 27 deletions

View File

@@ -24,6 +24,7 @@ and this project adheres to
- 🐛(frontend) fix broadcast store sync #1846 - 🐛(frontend) fix broadcast store sync #1846
- 🐛(helm) use celery resources instead of backend resources - 🐛(helm) use celery resources instead of backend resources
- 🐛(helm) reverse liveness and readiness for backend deployment
## [v4.5.0] - 2026-01-28 ## [v4.5.0] - 2026-01-28

View File

@@ -133,7 +133,6 @@ ingressCollaborationApi:
annotations: annotations:
nginx.ingress.kubernetes.io/upstream-hash-by: $arg_room nginx.ingress.kubernetes.io/upstream-hash-by: $arg_room
## @param ingressAdmin.enabled whether to enable the Ingress or not ## @param ingressAdmin.enabled whether to enable the Ingress or not
## @param ingressAdmin.className IngressClass to use for the Ingress ## @param ingressAdmin.className IngressClass to use for the Ingress
## @param ingressAdmin.host Host for the Ingress ## @param ingressAdmin.host Host for the Ingress
@@ -144,7 +143,7 @@ ingressAdmin:
host: impress.example.com host: impress.example.com
path: /admin path: /admin
## @param ingressAdmin.hosts Additional host to configure for the Ingress ## @param ingressAdmin.hosts Additional host to configure for the Ingress
hosts: [ ] hosts: []
# - chart-example.local # - chart-example.local
## @param ingressAdmin.tls.enabled Weather to enable TLS for the Ingress ## @param ingressAdmin.tls.enabled Weather to enable TLS for the Ingress
## @param ingressAdmin.tls.secretName Secret name for TLS config ## @param ingressAdmin.tls.secretName Secret name for TLS config
@@ -166,7 +165,7 @@ ingressMedia:
host: impress.example.com host: impress.example.com
path: /media/(.*) path: /media/(.*)
## @param ingressMedia.hosts Additional host to configure for the Ingress ## @param ingressMedia.hosts Additional host to configure for the Ingress
hosts: [ ] hosts: []
# - chart-example.local # - chart-example.local
## @param ingressMedia.tls.enabled Weather to enable TLS for the Ingress ## @param ingressMedia.tls.enabled Weather to enable TLS for the Ingress
## @param ingressMedia.tls.secretName Secret name for TLS config ## @param ingressMedia.tls.secretName Secret name for TLS config
@@ -197,11 +196,9 @@ serviceMedia:
port: 9000 port: 9000
annotations: {} annotations: {}
## @section backend ## @section backend
backend: backend:
## @param backend.command Override the backend container command ## @param backend.command Override the backend container command
command: [] command: []
@@ -231,7 +228,7 @@ backend:
allowPrivilegeEscalation: false allowPrivilegeEscalation: false
capabilities: capabilities:
drop: drop:
- "ALL" - "ALL"
runAsNonRoot: true runAsNonRoot: true
seccompProfile: seccompProfile:
type: RuntimeDefault type: RuntimeDefault
@@ -336,10 +333,10 @@ backend:
## @param backend.probes.readiness.initialDelaySeconds [nullable] Configure timeout for backend readiness probe ## @param backend.probes.readiness.initialDelaySeconds [nullable] Configure timeout for backend readiness probe
probes: probes:
liveness: liveness:
path: /__heartbeat__ path: /__lbheartbeat__
initialDelaySeconds: 10 initialDelaySeconds: 10
readiness: readiness:
path: /__lbheartbeat__ path: /__heartbeat__
initialDelaySeconds: 10 initialDelaySeconds: 10
## @param backend.resources Resource requirements for the backend container ## @param backend.resources Resource requirements for the backend container
@@ -381,7 +378,6 @@ backend:
file_content: "" file_content: ""
mount_path: /app/impress/configuration/theme mount_path: /app/impress/configuration/theme
## @param backend.celery.replicas Amount of celery replicas ## @param backend.celery.replicas Amount of celery replicas
## @param backend.celery.command Override the celery container command ## @param backend.celery.command Override the celery container command
## @param backend.celery.args Override the celery container args ## @param backend.celery.args Override the celery container args
@@ -395,22 +391,40 @@ backend:
celery: celery:
replicas: 1 replicas: 1
command: [] command: []
args: ["celery", "-A", "impress.celery_app", "worker", "-l", "INFO", "-n", "impress@%h"] args:
[
"celery",
"-A",
"impress.celery_app",
"worker",
"-l",
"INFO",
"-n",
"impress@%h",
]
resources: {} resources: {}
probes: probes:
liveness: liveness:
exec: exec:
command: ["/bin/sh", "-c", "celery -A impress.celery_app inspect ping -d impress@$HOSTNAME"] command:
[
"/bin/sh",
"-c",
"celery -A impress.celery_app inspect ping -d impress@$HOSTNAME",
]
initialDelaySeconds: 60 initialDelaySeconds: 60
timeoutSeconds: 5 timeoutSeconds: 5
readiness: readiness:
exec: exec:
command: ["/bin/sh", "-c", "celery -A impress.celery_app inspect ping -d impress@$HOSTNAME"] command:
[
"/bin/sh",
"-c",
"celery -A impress.celery_app inspect ping -d impress@$HOSTNAME",
]
initialDelaySeconds: 15 initialDelaySeconds: 15
timeoutSeconds: 5 timeoutSeconds: 5
## @section frontend ## @section frontend
frontend: frontend:
@@ -442,7 +456,7 @@ frontend:
allowPrivilegeEscalation: false allowPrivilegeEscalation: false
capabilities: capabilities:
drop: drop:
- "ALL" - "ALL"
runAsNonRoot: true runAsNonRoot: true
seccompProfile: seccompProfile:
type: RuntimeDefault type: RuntimeDefault
@@ -522,7 +536,6 @@ frontend:
## @section posthog ## @section posthog
posthog: posthog:
## @param posthog.ingress.enabled Enable or disable the ingress resource creation ## @param posthog.ingress.enabled Enable or disable the ingress resource creation
## @param posthog.ingress.className Kubernetes ingress class name to use (e.g., nginx, traefik) ## @param posthog.ingress.className Kubernetes ingress class name to use (e.g., nginx, traefik)
## @param posthog.ingress.host Primary hostname for the ingress resource ## @param posthog.ingress.host Primary hostname for the ingress resource
@@ -537,12 +550,12 @@ posthog:
className: null className: null
host: impress.example.com host: impress.example.com
path: / path: /
hosts: [ ] hosts: []
tls: tls:
enabled: true enabled: true
additional: [ ] additional: []
customBackends: [ ] customBackends: []
annotations: {} annotations: {}
## @param posthog.ingressAssets.enabled Enable or disable the ingress resource creation ## @param posthog.ingressAssets.enabled Enable or disable the ingress resource creation
@@ -559,14 +572,14 @@ posthog:
className: null className: null
host: impress.example.com host: impress.example.com
paths: paths:
- /static - /static
- /array - /array
hosts: [ ] hosts: []
tls: tls:
enabled: true enabled: true
additional: [ ] additional: []
customBackends: [ ] customBackends: []
annotations: {} annotations: {}
## @param posthog.service.type Service type (e.g. ExternalName, ClusterIP, LoadBalancer) ## @param posthog.service.type Service type (e.g. ExternalName, ClusterIP, LoadBalancer)
@@ -589,7 +602,6 @@ posthog:
port: 443 port: 443
annotations: {} annotations: {}
## @section yProvider ## @section yProvider
yProvider: yProvider:
@@ -621,7 +633,7 @@ yProvider:
allowPrivilegeEscalation: false allowPrivilegeEscalation: false
capabilities: capabilities:
drop: drop:
- "ALL" - "ALL"
runAsNonRoot: true runAsNonRoot: true
seccompProfile: seccompProfile:
type: RuntimeDefault type: RuntimeDefault
@@ -729,7 +741,7 @@ docSpec:
allowPrivilegeEscalation: false allowPrivilegeEscalation: false
capabilities: capabilities:
drop: drop:
- "ALL" - "ALL"
runAsNonRoot: true runAsNonRoot: true
seccompProfile: seccompProfile:
type: RuntimeDefault type: RuntimeDefault