helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

[Feature Request] Enable LivenessProbe for workers

Open gracco opened this issue 3 years ago • 0 comments

Is your feature request related to a problem? Please describe.

When we deploy a new cluster, the workers nodes don't have a LivenessProbe because it's explicit configured in the code to not add them.

Describe the solution you'd like

Remove the if loop in the lines above. Or implement a specific livenessProbe for workers.

https://github.com/temporalio/helm-charts/blob/master/templates/server-deployment.yaml#L127-L132

{{- if ne $service "worker"}}
          livenessProbe:
             initialDelaySeconds: 150
             tcpSocket:
               port: rpc
 {{- end }}

Additional context

gracco avatar Jun 08 '22 17:06 gracco