Skip to content

Commit 58057a2

Browse files
authored
config: Fix readiness probe causing ~3m startup delay (#610)
PR #555 changed readiness probe periodSeconds from 10 to 180 to anticipate big clusters with 10K VMs. However, periodSeconds controls how often Kubernetes checks the probe, not how long to tolerate initialization. With periodSeconds=180, if the first probe at T=10s fails (e.g. webhook server not yet serving), the next attempt is at T=190s, leaving the pod not-Ready for over 3 minutes. Restore periodSeconds to 10s. The pool manager's /readyz endpoint already handles long initialization correctly — it returns failure until IsReady() is true, and Kubernetes keeps probing every periodSeconds until it succeeds. Assisted-by: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Ram Lavi <ralavi@redhat.com>
1 parent 0c32241 commit 58057a2

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

config/default/manager/manager.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ spec:
136136
port: webhook-server
137137
scheme: HTTPS
138138
initialDelaySeconds: 10
139-
periodSeconds: 180
139+
periodSeconds: 10
140140
livenessProbe:
141141
httpGet:
142142
httpHeaders:

config/release/kubemacpool.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@ spec:
329329
port: webhook-server
330330
scheme: HTTPS
331331
initialDelaySeconds: 10
332-
periodSeconds: 180
332+
periodSeconds: 10
333333
resources:
334334
requests:
335335
cpu: 100m

config/test/kubemacpool.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -330,7 +330,7 @@ spec:
330330
port: webhook-server
331331
scheme: HTTPS
332332
initialDelaySeconds: 10
333-
periodSeconds: 180
333+
periodSeconds: 10
334334
resources:
335335
requests:
336336
cpu: 100m

0 commit comments

Comments
 (0)