-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Propagate downstream timeout to upstream through multiple Envoys #7358
Comments
Yeah agreed I think either would solve. I don't have a super strong opinion on which one. This does bring up the general topic of deadline propagation which we haven't really tackled yet in a holistic way. I've thought at some point we may also want to stick the deadline in trace context baggage since this would also propagate through app calls, though that is a larger problem than you are trying to solve here. |
+1. It would be very useful. Inserting |
This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or other activity occurs. Thank you for your contributions. |
@snowp would like to help out with this. |
@nezdolik Great, I'll assign this one to you. Feel free to ping me with any questions |
When running Envoy on both egress and ingress, the client will provide a timeout header to the egress Envoy, which will propagate the expected upstream timeout in
x-envoy-expected-rq-timeout-ms
. The upstream Envoy will not read this header, so it will resolve a new timeout value that is set as the expected timeout for the upstream service. This means that the deadline expected by the egress Envoy is ignored in favor of the ingress Envoy, resulting in the upstream service having an incorrect view of the actual deadline.It seems like either
x-envoy-rq-timeout-ms
with the expected timeout on egressx-envoy-expected-rq-timeout-ms
as the deadline on the ingress sidewould solve the issue (likely guarded by a config flag).
Thoughts?
The text was updated successfully, but these errors were encountered: