-
Notifications
You must be signed in to change notification settings - Fork 20
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
perf: add liveness and readiness probes to deployments #85
Conversation
67d3d09
to
555c512
Compare
4e9a620
to
52b16e8
Compare
9362b46
to
0d28729
Compare
@anishasthana @KPostOffice can one of ye give me a lgtm here so I can merge please? |
kind bump to review this PR :) |
do we know what is pending for getting this PR merged? |
@KPostOffice sorry to poke again, but can you approve this? |
Can we please make sure we have manually tested this PR? |
@dimakis do we know what is pending on this PR? |
22acb43
to
db3b69e
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: astefanutti The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
linked here in the codeflare operator