-
Notifications
You must be signed in to change notification settings - Fork 82
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
Use default gRPC readiness probe #981
Use default gRPC readiness probe #981
Conversation
Codecov Report
@@ Coverage Diff @@
## main #981 +/- ##
=======================================
Coverage 80.78% 80.78%
=======================================
Files 18 18
Lines 1343 1343
=======================================
Hits 1085 1085
Misses 205 205
Partials 53 53 |
/hold |
@nak3 is this ready? |
This will be a breaking change for the old k8s cluster (or OCP) so we will not merge until we are sure that the gRPC readiness probe is available. (after 1.10 or 1.11 release.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I like it.
/lgtm
/hold for other reviewers to chime in.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nak3, ReToCode 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 |
As per title, this patch changes to use default gRPC readiness probe.
Additional Note
Current timeout is 100 milli seconds but
timeoutSeconds
is minimum 1 second so this change will make the timeout longer than current.Fix #923