Add default proxy startupProbe parameters to values.yaml

Signed-off-by: TJ Miller <[email protected]>

Author: teejaded

charts/linkerd-control-plane/README.md

@@ -255,6 +255,7 @@ Kubernetes: `>=1.21.0-0`
 | proxy.resources.memory.limit | string | `""` | Maximum amount of memory that the proxy can use |
 | proxy.resources.memory.request | string | `""` | Maximum amount of memory that the proxy requests |
 | proxy.shutdownGracePeriod | string | `""` | Grace period for graceful proxy shutdowns. If this timeout elapses before all open connections have completed, the proxy will terminate forcefully, closing any remaining connections. |
+| proxy.startupProbe | object | `{"failureThreshold":120,"initialDelaySeconds":0,"periodSeconds":1}` | Native sidecar proxy startup probe parameters. |
 | proxy.uid | int | `2102` | User id under which the proxy runs |
 | proxy.waitBeforeExitSeconds | int | `0` | If set the injected proxy sidecars in the data plane will stay alive for at least the given period before receiving the SIGTERM signal from Kubernetes but no longer than the pod's `terminationGracePeriodSeconds`. See [Lifecycle hooks](https://kubernetes.io/docs/concepts/containers/container-lifecycle-hooks/#container-hooks) for more info on container lifecycle hooks. |
 | proxyInit.closeWaitTimeoutSecs | int | `0` |  |

charts/linkerd-control-plane/values.yaml

@@ -195,6 +195,11 @@ proxy:
   # This is an experimental feature. It requires Kubernetes >= 1.29.
   # If enabled, .proxy.waitBeforeExitSeconds should not be used.
   nativeSidecar: false
+  # -- Native sidecar proxy startup probe parameters.
+  startupProbe:
+    initialDelaySeconds: 0
+    periodSeconds: 1
+    failureThreshold: 120
 
 # proxy-init configuration
 proxyInit:

charts/partials/templates/_proxy.tpl

@@ -176,9 +176,9 @@ startupProbe:
   httpGet:
     path: /ready
     port: {{.Values.proxy.ports.admin}}
-  initialDelaySeconds: {{.Values.proxy.startupProbeInitialDelaySeconds | default 0}}
-  periodSeconds: {{.Values.proxy.startupProbePeriodSeconds | default 1}}
-  failureThreshold: {{.Values.proxy.startupProbeFailureThreshold | default 120}}
+  initialDelaySeconds: {{.Values.proxy.startupProbe.initialDelaySeconds}}
+  periodSeconds: {{.Values.proxy.startupProbe.periodSeconds}}
+  failureThreshold: {{.Values.proxy.startupProbe.failureThreshold}}
 {{- end }}
 {{- if .Values.proxy.resources }}
 {{ include "partials.resources" .Values.proxy.resources }}

cli/cmd/testdata/install_controlplane_tracing_output.golden

@@ -120,6 +120,7 @@ type (
 		AccessLog                            string           `json:"accessLog"`
 		ShutdownGracePeriod                  string           `json:"shutdownGracePeriod"`
 		NativeSidecar                        bool             `json:"nativeSidecar"`
+		StartupProbe                         *StartupProbe    `json:"startupProbe"`
 	}
 
 	// ProxyInit contains the fields to set the proxy-init container
@@ -227,6 +228,13 @@ type (
 		EphemeralStorage Constraints `json:"ephemeral-storage"`
 	}
 
+	// StartupProbe represents the initContainer startup probe parameters for the proxy
+	StartupProbe struct {
+		InitialDelaySeconds uint `json:"initialDelaySeconds"`
+		PeriodSeconds       uint `json:"periodSeconds"`
+		FailureThreshold    uint `json:"failureThreshold"`
+	}
+
 	// Identity contains the fields to set the identity variables in the proxy
 	// sidecar container
 	Identity struct {

cli/cmd/testdata/install_custom_domain.golden

@@ -134,6 +134,11 @@ func TestNewValues(t *testing.T) {
 			InboundDiscoveryCacheUnusedTimeout:   "90s",
 			DisableOutboundProtocolDetectTimeout: false,
 			DisableInboundProtocolDetectTimeout:  false,
+			StartupProbe: &StartupProbe{
+				FailureThreshold:    120,
+				InitialDelaySeconds: 0,
+				PeriodSeconds:       1,
+			},
 		},
 		ProxyInit: &ProxyInit{
 			IptablesMode:        "legacy",