Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setup Readiness and Liveness probe #4732

Closed
phisco opened this issue Oct 5, 2023 · 2 comments
Closed

Setup Readiness and Liveness probe #4732

phisco opened this issue Oct 5, 2023 · 2 comments
Labels
bug Something isn't working e2e enhancement New feature or request
Milestone

Comments

@phisco
Copy link
Contributor

phisco commented Oct 5, 2023

What happened?

Crossplane currently doesn't have any liveness probe configured, with webhooks enabled this could lead to some calls being rejected at Crossplane startup, which is undesirable.

This is not really a big issue, as those calls would still fail on first installation, but could improve the upgrade process, as the new pod would only be considered ready once it's actually ready to serve traffic, properly leveraging Kubernetes rollout.

It would be nice if we could add probes to Crossplane's pod, see the conversation here for some hints at how to do that.

How can we reproduce it?

It's a timing issue, just spinning up Crossplane while applying any resource having a validating webhook, so either a Composition or CompositeResourceDefinition, could lead to some failure:

failed calling webhook "compositions.apiextensions.crossplane.io": failed to call webhook: Post "https://crossplane-webhooks.crossplane-system.svc:9443/validate-apiextensions-crossplane-io-v1-composition?timeout=10s": dial tcp 10.96.47.154:9443: connect: connection refused

What environment did it happen in?

Crossplane version: latest master, but most probably happening since 1.11

@phisco phisco added bug Something isn't working enhancement New feature or request labels Oct 5, 2023
@negz
Copy link
Member

negz commented Oct 5, 2023

FWIW I've hit the above as a flaky E2E test a few times now.

@negz negz added the e2e label Oct 5, 2023
@phisco phisco closed this as completed Oct 16, 2023
@phisco
Copy link
Contributor Author

phisco commented Oct 16, 2023

Fixed by #4748

@jbw976 jbw976 added this to the v1.14 milestone Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working e2e enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

3 participants