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.

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

grpc-js-xds: Add interop server implementation #2897

Merged
merged 18 commits into from
Feb 5, 2025

Conversation

murgatroid99
Copy link
Member

@murgatroid99 murgatroid99 commented Feb 4, 2025

This includes the interop server implementation code itself, the corresponding Dockerfile, and the test script changes. In addition, I made a couple of small changes to the library code:

  • I changed the tracer names of a couple of events in the server code to consistently separate per-call tracers and non-per-call tracers.
  • I fixed a bug that was causing the sendMetadata method in server interceptors to not be called if the handler function did not explicitly send metadata.

This change is tested with the corresponding interop driver change at grpc/node/master/xds_k8s_lb and grpc/node/master/psm-dualstack.

@murgatroid99
Copy link
Member Author

murgatroid99 commented Feb 4, 2025

@murgatroid99 murgatroid99 force-pushed the grpc-js-xds_interop_server branch from d5ea254 to 8fea44d Compare February 5, 2025 17:53
@murgatroid99
Copy link
Member Author

murgatroid99 commented Feb 5, 2025

@murgatroid99 murgatroid99 merged commit 06a05a4 into grpc:master Feb 5, 2025
8 of 10 checks passed
murgatroid99 added a commit to grpc/psm-interop that referenced this pull request Feb 5, 2025
This modifies the driver to use the Node interop server implemented in
grpc/grpc-node#2897 whenever possible. Test
runs:

- [x]
[grpc/node/master/xds_k8s_lb](https://source.cloud.google.com/results/invocations/5eda22ed-1bf6-4a9e-b7ee-7da93c2dfa09)
- [ ]
~[grpc/node/master/psm-dualstack](https://source.cloud.google.com/results/invocations/407b7ba1-ce14-4e3d-a025-b9219fffb3e1)~
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.

1 participant