Skip to content

Added readinessProbe to webhook deployment#287

Open
sotoiwa wants to merge 2 commits into
aws:masterfrom
sotoiwa:patch-readinessProbe
Open

Added readinessProbe to webhook deployment#287
sotoiwa wants to merge 2 commits into
aws:masterfrom
sotoiwa:patch-readinessProbe

Conversation

@sotoiwa

@sotoiwa sotoiwa commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

This PR adds a readinessProbe to prevent traffic from being routed to webhook pods before they're ready to handle requests.

Problem

Without readinessProbe, webhook pods receive traffic immediately after startup, which can cause "connection refused" errors during cluster operations (upgrades, rolling updates). Due to failurePolicy: Ignore, pods are created without expected IAM role mutations.

Solution

Added readinessProbe using the existing /healthz endpoint:

        livenessProbe:
          httpGet:
            path: /healthz
            port: 443
            scheme: HTTPS
          initialDelaySeconds: 10
          periodSeconds: 10
          failureThreshold: 3
        readinessProbe:
          httpGet:
            path: /healthz
            port: 443
            scheme: HTTPS
          initialDelaySeconds: 5
          periodSeconds: 5
          failureThreshold: 2

Related Issues

While the current /healthz endpoint is basic, it ensures the webhook server is at least accepting connections before receiving traffic, significantly improving reliability during cluster operations.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sotoiwa sotoiwa requested a review from a team as a code owner December 1, 2025 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Specify liveness & readiness probes for the pod-identity-webhook

1 participant