Skip to content

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