-
Notifications
You must be signed in to change notification settings - Fork 27
Add gRPC client and worker connection resiliency #135
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
Merged
Merged
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
8212945
Add gRPC resiliency design spec
8b25b56
Ignore local worktrees
fd07d1a
Add gRPC resiliency implementation plan
774004a
Add gRPC resiliency option types
4e8d0bb
Add grpc resiliency validation tests
c09def6
Thread gRPC resiliency options through constructors
a6bc855
Strengthen retained client state tests
a2ff52b
Add shared gRPC resiliency helpers
c049899
Add completion long-poll resiliency test
19e3d71
Add grpc resiliency edge-case tests
008be65
Harden worker gRPC stream reconnect behavior
c4a98e9
Fix worker channel cleanup on teardown
950892b
Add worker silent disconnect tests
09697bb
Add sync client gRPC channel recreation
d89bc10
Reset sync client long-poll failure tracking
937231d
Add sync client recreation input test
2834177
Add sync client recreation test coverage
4f4b3a3
Add async client gRPC channel recreation
762b247
Add async channel recreation transport test
2363926
Add gRPC connection resiliency
2401224
Remove repo-wide pytest importlib addopts
bbaf2b0
Update gRPC resiliency plan tracking
72191f1
Fix worker channel retirement for in-flight completions
a49f5ce
Fix worker shutdown channel draining
32f383d
Rename Azure Managed gRPC resiliency test module
b6f3b43
Fix sync client channel cleanup
1f9545f
Address automated review feedback
bdb37e2
Remove superpowers docs from PR
78829a0
Merge branch 'main' into grpc-resiliency-pr708-impl
andystaples 345d941
Address andystaples PR review feedback
f2fa6d3
Refactor client resiliency to use UnaryUnaryClientInterceptor
223ba40
Merge remote-tracking branch 'origin/main' into berndverst/pr-135-fee…
935503f
Use inspect.isawaitable in AsyncClientResiliencyInterceptor
053b579
Discard awaitable result with explicit underscore in resiliency inter…
3268b45
Fire-and-forget channel recreate; hoist client state; tighten async i…
845c697
Modernize PR-new code for Python 3.10+ baseline
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -131,5 +131,7 @@ dmypy.json | |
|
|
||
| # IDEs | ||
| .idea | ||
| .worktrees/ | ||
| docs/superpowers/ | ||
|
|
||
| coverage.lcov | ||
| coverage.lcov | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.