From cf50da8e5aea8f89ff995ae6a521bc0f93dd3266 Mon Sep 17 00:00:00 2001 From: zhenyus Date: Thu, 31 Jul 2025 23:58:53 +0800 Subject: [PATCH] fix(reconciler): simplify health check probe configuration in deployment template - Updated liveness and readiness probe configurations to directly reference the HTTP GET settings from the values, improving clarity and reducing unnecessary nesting. - Ensures that the deployment template remains consistent and easier to maintain. Signed-off-by: zhenyus --- .../helm-pkg/reconciler/templates/deployment.yaml | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/freeleaps-devops-reconciler/helm-pkg/reconciler/templates/deployment.yaml b/freeleaps-devops-reconciler/helm-pkg/reconciler/templates/deployment.yaml index 1a41fda0..89bc5ca2 100644 --- a/freeleaps-devops-reconciler/helm-pkg/reconciler/templates/deployment.yaml +++ b/freeleaps-devops-reconciler/helm-pkg/reconciler/templates/deployment.yaml @@ -43,10 +43,8 @@ spec: {{- include "freeleaps-devops-reconciler.env" . | nindent 12 }} {{- if .Values.healthcheck.livenessProbe.enabled }} livenessProbe: - {{- with .Values.healthcheck.livenessProbe.httpGet }} httpGet: - {{- toYaml . | nindent 14 }} - {{- end }} + {{- toYaml .Values.healthcheck.livenessProbe.httpGet | nindent 14 }} initialDelaySeconds: {{ .Values.healthcheck.livenessProbe.initialDelaySeconds }} periodSeconds: {{ .Values.healthcheck.livenessProbe.periodSeconds }} timeoutSeconds: {{ .Values.healthcheck.livenessProbe.timeoutSeconds }} @@ -54,10 +52,8 @@ spec: {{- end }} {{- if .Values.healthcheck.readinessProbe.enabled }} readinessProbe: - {{- with .Values.healthcheck.readinessProbe.httpGet }} httpGet: - {{- toYaml . | nindent 14 }} - {{- end }} + {{- toYaml .Values.healthcheck.readinessProbe.httpGet | nindent 14 }} initialDelaySeconds: {{ .Values.healthcheck.readinessProbe.initialDelaySeconds }} periodSeconds: {{ .Values.healthcheck.readinessProbe.periodSeconds }} timeoutSeconds: {{ .Values.healthcheck.readinessProbe.timeoutSeconds }}