[backport v0.31.x-gp] custom ReadinessProbe (#483) (#484)

* custom ReadinessProbe (#483)

* custom ReadinessProbe

* merge

* typo

* comment

* changelog type fix

Author: puertomontt

changelog/v0.31.7/readiness-probe.yaml

@@ -0,0 +1,5 @@
+changelog:
+  - type: FIX
+    issueLink: https://github.com/solo-io/gloo-mesh-enterprise/issues/11314
+    resolvesIssue: false
+    description: Use custom type for readiness probe to allow passing in templates for values.

codegen/cmd_test.go

@@ -781,13 +781,9 @@ var _ = Describe("Cmd", func() {
 										Value: "BAR",
 									},
 								},
-								ReadinessProbe: &v1.Probe{
-									ProbeHandler: v1.ProbeHandler{
-										HTTPGet: &v1.HTTPGetAction{
-											Path: "/",
-											Port: intstr.FromInt(8080),
-										},
-									},
+								ReadinessProbe: &ReadinessProbe{
+									Path:                "/",
+									Port:                "8080",
 									PeriodSeconds:       10,
 									InitialDelaySeconds: 5,
 								},
@@ -944,13 +940,9 @@ var _ = Describe("Cmd", func() {
 										Value: "BAR",
 									},
 								},
-								ReadinessProbe: &v1.Probe{
-									ProbeHandler: v1.ProbeHandler{
-										HTTPGet: &v1.HTTPGetAction{
-											Path: "/",
-											Port: intstr.FromInt(8080),
-										},
-									},
+								ReadinessProbe: &ReadinessProbe{
+									Path:                "/",
+									Port:                "8080",
 									PeriodSeconds:       10,
 									InitialDelaySeconds: 5,
 								},
@@ -1532,13 +1524,8 @@ roleRef:
 											Value: "BAR",
 										},
 									},
-									ReadinessProbe: &v1.Probe{
-										ProbeHandler: v1.ProbeHandler{
-											HTTPGet: &v1.HTTPGetAction{
-												Path: "/",
-												Port: intstr.FromInt(8080),
-											},
-										},
+									ReadinessProbe: &ReadinessProbe{
+										Exec:                []string{"redis-cli", "ping"},
 										PeriodSeconds:       10,
 										InitialDelaySeconds: 5,
 									},

codegen/model/chart.go

@@ -124,7 +124,7 @@ type Container struct {
 	// not configurable via helm values
 	Args           []string
 	VolumeMounts   []corev1.VolumeMount
-	ReadinessProbe *corev1.Probe
+	ReadinessProbe *ReadinessProbe
 	LivenessProbe  *corev1.Probe
 
 	Image           Image
@@ -133,6 +133,14 @@ type Container struct {
 	SecurityContext *corev1.SecurityContext
 }
 
+type ReadinessProbe struct {
+	Exec                []string // optional: if specified, the readiness probe will be an exec probe with the specified commands
+	Path                string   // Path to access on the HTTP server. Either specify Path and Port for httpGet probes, or specify Exec
+	Port                string
+	PeriodSeconds       int
+	InitialDelaySeconds int
+}
+
 // sidecars require a container config and a unique name
 type Sidecar struct {
 	Container

codegen/templates/chart/operator-deployment.yamltmpl

@@ -151,7 +151,23 @@ spec:
 {{- end }}
 [[- if $container.ReadinessProbe ]]
         readinessProbe:
-[[ toYaml $container.ReadinessProbe | indent 10 ]]
+        [[- if $container.ReadinessProbe.Exec ]]
+          exec:
+            command:
+            [[- range $command := $container.ReadinessProbe.Exec ]]
+            - [[ $command ]]
+            [[- end ]]
+        [[- else ]]
+          httpGet:
+            path: [[ $container.ReadinessProbe.Path ]]
+            port: [[ $container.ReadinessProbe.Port ]]
+        [[- end ]]
+        [[- if $container.ReadinessProbe.InitialDelaySeconds ]]
+          initialDelaySeconds: [[ $container.ReadinessProbe.InitialDelaySeconds ]]
+        [[- end ]]
+        [[- if $container.ReadinessProbe.PeriodSeconds ]]
+          periodSeconds: [[ $container.ReadinessProbe.PeriodSeconds ]]
+        [[- end ]]
 [[- end ]]
 [[- if $container.LivenessProbe ]]
         livenessProbe:

codegen/test/chart-no-desc/templates/deployment.yaml

@@ -90,9 +90,10 @@ spec:
             - ALL
 {{- end }}
         readinessProbe:
-          httpGet:
-            path: /
-            port: 8080
+          exec:
+            command:
+            - redis-cli
+            - ping
           initialDelaySeconds: 5
           periodSeconds: 10
 {{- $palette := $.Values.painter.sidecars.palette }}