Skip to content

Add gRPC integration tests for http relay and update docs#684

Merged
dfreese merged 3 commits into
googlecloudrobotics:mainfrom
dfreese:freese/streaming
May 19, 2026
Merged

Add gRPC integration tests for http relay and update docs#684
dfreese merged 3 commits into
googlecloudrobotics:mainfrom
dfreese:freese/streaming

Conversation

@dfreese
Copy link
Copy Markdown
Contributor

@dfreese dfreese commented May 18, 2026

This adds an integration test that demonstrates the use of different
gRPC types over the http relay (unary, server-streaming,
client-streaming, and bi-directional streaming). I have updated the
documentation to reflect that this works. I was never able to reproduce
the specific failure mentioned in the docs. From my other experiments,
server-streaming was working without issue, so I think this integration
test is representative of the current state. I haven't attempted to
trace if this was fixed by a particular commit.

This adds an integration test that demonstrates the use of different
gRPC types over the http relay (unary, server-streaming,
client-streaming, and bi-directional streaming).  I have updated the
documentation to reflect that this works.  I was never able to reproduce
the specific failure mentioned in the docs.  From my other experiments,
server-streaming was working without issue, so I think this integration
test is representative of the current state.  I haven't attempted to
trace if this was fixed by a particular commit.
@dfreese dfreese force-pushed the freese/streaming branch from 3bfed45 to 7c2a4fa Compare May 18, 2026 21:38
@dfreese dfreese changed the title Add server-streaming gRPC integration test Add gRPC integration tests for http relay and update docs May 18, 2026
Copy link
Copy Markdown
Contributor

@drigz drigz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is really nice, thank you so much!

Comment thread src/go/tests/relay/grpc_test.go Outdated
Comment thread src/go/tests/relay/grpc_test.go
Comment thread src/go/tests/relay/grpc_test.go
Comment thread src/go/tests/relay/grpc_test.go Outdated
Comment thread src/go/tests/relay/grpc_test.go Outdated
Copy link
Copy Markdown
Contributor

@ensonic ensonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@dfreese dfreese enabled auto-merge (squash) May 19, 2026 19:23
@dfreese dfreese merged commit 26f503b into googlecloudrobotics:main May 19, 2026
8 of 10 checks passed
@dfreese dfreese deleted the freese/streaming branch May 19, 2026 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants