Skip to content

Support gRPC 1.82 and fix test parallelism - #175

Merged
riz0id merged 9 commits into
masterfrom
tomas/bump-nixpkgs
Jul 28, 2026
Merged

Support gRPC 1.82 and fix test parallelism#175
riz0id merged 9 commits into
masterfrom
tomas/bump-nixpkgs

Conversation

@tm-drtina

Copy link
Copy Markdown
Contributor

Summary

Update tests to work with gRPC 1.82+, which enables the error_flatten experiment
that drops status messages for OK status (matching absl::Status semantics).
Also fix test failures caused by Tasty 1.5.4 enabling parallel execution by default.

Changes

gRPC 1.82 compatibility

  • Expose grpcCppVersionMajor / grpcCppVersionMinor from Network.GRPC.Unsafe
    via C helpers that read GRPC_CPP_VERSION_MAJOR / GRPC_CPP_VERSION_MINOR
    from grpcpp/version_info.h (since grpc_version_string() returns the
    internal core version, not the release version)
  • Use runtime version check in core tests to conditionally expect empty status
    details for OK responses on gRPC >= 1.82

Test parallelism fixes

  • Tasty 1.5.4 makes -j entail +RTS -N automatically, which causes tests
    to run in parallel by default
  • grpc-haskell-core: Set NumThreads 1 since all network tests share port 50051
  • grpc-haskell: Wrap generated code tests in dependentTestGroup AllFinish
    since they share port 50051 and temp directory tests/tmp

Comment thread core/src/Network/GRPC/Unsafe.chs Outdated
Comment thread core/include/grpc_haskell.h Outdated
@tm-drtina
tm-drtina requested a review from riz0id July 28, 2026 16:20
@riz0id
riz0id merged commit 0e16723 into master Jul 28, 2026
8 checks passed
@riz0id
riz0id deleted the tomas/bump-nixpkgs branch July 28, 2026 22:00
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.

2 participants