-
Notifications
You must be signed in to change notification settings - Fork 12
Add a gRPC wire beside the websocket for inference sessions #728
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
Open
v-positronic
wants to merge
46
commits into
main
Choose a base branch
from
grpc-wire
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
cd048df
Add a gRPC wire beside the websocket for inference sessions
v-positronic c2487b3
Open the endpoint-description line on its subject
v-positronic deba93f
Tighten two comments the writing gate named
v-positronic b73a933
Bracket an IPv6 host in the address the gRPC wire binds
v-positronic 4ab68e4
Pin the frame ceiling for both wires, and close a connection the hand…
v-positronic be5921e
Carry a gRPC session through a TLS edge and a front's idle close
v-positronic 6155128
Add a wire-only stub server for measuring a session's transport
v-positronic f6028f2
Serve both wires from a Nebius endpoint, and say what its front does
v-positronic 58292e4
Say the front's contract once, where the README already holds it
v-positronic a5aa9aa
Put the subject first in two sentences about the TLS edge
v-positronic 859fecd
Read a TLS edge no client can use as permanent, and expose the port a…
v-positronic 9904200
State the ping defaults without counting them first
v-positronic 267a3a9
Say why the websocket is not shaped, rather than closing on the sound…
v-positronic cc26f7d
Leave the front's idle close to the README, and name the constant's job
v-positronic 4e9fa75
Refuse a closed gRPC session, and drop the credentials alias
v-positronic bb1d888
State the session-param constraint without weighing the alternative
v-positronic b4bc5da
Refuse a send into an ended stream, and pin the websocket's own ceiling
v-positronic f871822
Name the subject first in two comments about an ended session
v-positronic 3b24ad7
Keep the connect refusal probe inside one attempt's budget
v-positronic d93e06a
Type the URL scheme a session opens on
v-positronic 931109a
Name the websocket port once in the serve script
v-positronic 3c26716
Name a remote session for what it is, not for one wire it can take
v-positronic 030eba2
Name the compress-images metadata key once in its test
v-positronic deffec0
Cover the close report on both wires, and place two helpers above the…
v-positronic f710044
Hold a test server's websocket port from the draw to the serve
v-positronic 4907e2e
Keep a short open timeout usable, and bind the gRPC test wire on any …
v-positronic 569d4dc
Say the port rule without listing the TLS schemes
v-positronic eb26d62
Say which port the serve script fixes and which the caller names
v-positronic 0b9ab74
List the gRPC schemes where the eval docs name what a policy URL takes
v-positronic c84de5c
Name the session protocol rather than one of its wires in the eval docs
v-positronic a45453f
Take the wires sessions arrive on, so the server names none of them
v-positronic 14a2c38
Report every wire that failed, and put two constants beside their users
v-positronic 7160990
Say what `PolicyServer` owns, and leave the wire contract to `Wire`
v-positronic 9c00138
Drive the vendor server tests through the session handler
v-positronic 43536c7
Stop the wires that bound when a later one cannot
v-positronic 55bb7f4
Close the websocket wire's socket when startup rolls back
v-positronic 811c04c
Move the websocket wire into its own module
v-positronic da23946
Cut the comments and docs of the wires to what holds
v-positronic 816cdc0
Cut two client comments to two lines and give `serve.sh` a usage block
v-positronic c01b1d3
State the README's protocol claims as facts
v-positronic 1a9a9d1
Rename `UNUSABLE_EDGE_DETAILS` and cut the inherited docstrings to wh…
v-positronic c9e66e2
State where the served model's values are named
v-positronic 9738f0c
Raise a wire-neutral refusal from each wire's dial
v-positronic 0f3f5b4
Report a lost peer when a status ends a started gRPC session
v-positronic b884a13
Log an unexpected error delivery failure at ERROR
v-positronic 735741f
Serve both wires in the offboard docs
v-positronic 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
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.