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.

Sign up for GitHub

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

Jump to bottom

Attach channel state information to locally generated DEADLINE_EXCEEDED errors #4740

Open
zhangkun83 opened this issue Aug 7, 2018 · 1 comment
Open

Attach channel state information to locally generated DEADLINE_EXCEEDED errors #4740

zhangkun83 opened this issue Aug 7, 2018 · 1 comment
Assignees
@zhangkun83
Labels
enhancement
Milestone
Next

Comments

@zhangkun83
Copy link
Contributor

Additional information can be added to description to help debug DEADLINE_EXCEEDED errors.

Things to consider:

  • Current connectivity state
  • Latest events from channel trace
  • Last error from LoadBalancer (saved even after LoadBalancer got out of TRANSIENT_FAILURE state)
@ejona86 ejona86 added this to the Next milestone Aug 14, 2018
@zhangkun83 zhangkun83 self-assigned this Aug 14, 2018
@ejona86
Copy link
Member

ejona86 commented Apr 18, 2019

This would have helped debugging a mobile user today.

This was referenced Jun 12, 2019
Closed
Merged
zhangkun83 added a commit that referenced this issue Jun 20, 2019
@zhangkun83
core: attach debug information about stream to DEADLINE_EXCEEDED (#5892)
ddbaf74 
Works for #4740 

- Subclasses of `AbstractClientStream` include remote address in insight if available.
- `DelayedStream` adds buffered time, and the insight of real stream if it's set.
- `RetriableStream` insights outputs of Substreams.

Example error message:
```
deadline exceeded after 8112071ns. [buffered_nanos=24763, remote_addr=foo.test.google.fr/127.0.0.1:44749]
```
or
```
deadline exceeded after 8112071ns. [buffered_nanos=22344324763, waiting_for_connection]
```

This is related to #4776 but taking a more usage-specific approach.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhangkun83 zhangkun83

Labels
enhancement
Projects
None yet
Milestone
Next
Development

No branches or pull requests
2 participants
@ejona86 @zhangkun83