-
Notifications
You must be signed in to change notification settings - Fork 615
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
[17.03] Increase gRPC request timeout to 20 seconds when sending snapshots #2406
[17.03] Increase gRPC request timeout to 20 seconds when sending snapshots #2406
Conversation
CI failure does not seem related, all the issues seem to be style/formatting issues. |
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.
My bad, I hit approve too soon.
CI failure does not seem related, all the issues seem to be style/formatting issues.
We'll need to port over the fixes for the lint failures:
aa2c48b
Lint errors fixed in #2418 |
(cherry picked from commit e3e2821) Signed-off-by: Nishant Totla <nishanttotla@gmail.com>
58ca4fa
to
9b67599
Compare
Codecov Report
@@ Coverage Diff @@
## bump_v17.03 #2406 +/- ##
===============================================
- Coverage 55.78% 55.43% -0.35%
===============================================
Files 102 102
Lines 21532 21544 +12
===============================================
- Hits 12011 11943 -68
- Misses 8368 8446 +78
- Partials 1153 1155 +2 |
LGTM |
I know @davidwilliamson tested this manually, but I am not sure there is an automated test for this. cc @antonybichon17 |
…17.03 [17.03] Increase gRPC request timeout to 20 seconds when sending snapshots
Cherry-pick #2391
I had to manually make these changes, since the cherry-pick wasn't clean. I think this needs a code review.
cc @anshulpundir @aaronlehmann @andrewhsu @thaJeztah
Signed-off-by: Nishant Totla nishanttotla@gmail.com