Skip to content

Commit d6631f5

Browse files
authored
[grpc storage]: Propagate tenant to grpc backend (jaegertracing#6030)
## Which problem is this PR solving? - Missing tenant information in a gRPC remote backend. ![image](https://github.com/user-attachments/assets/fea54fe0-5785-4225-a8cc-29b705a8be24) ## Description of the changes Before the gRPC plugins were removed in Jaeger 1.58, tenant information was distributed in via the internal context. With this change, the tenant information is also propagated to a removed grpc backend. ## How was this change tested? - Manually with jaeger-query, tempo-query and wireshark. ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --- cc @albertteoh do you think we can get this into the next jaeger release tomorrow? ^^ --------- Signed-off-by: Benedikt Bongartz <[email protected]>
1 parent d13b943 commit d6631f5

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

plugin/storage/grpc/config.go

+1
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ func DefaultConfigV2() ConfigV2 {
3838

3939
func (c *Configuration) TranslateToConfigV2() *ConfigV2 {
4040
return &ConfigV2{
41+
Tenancy: c.TenancyOpts,
4142
ClientConfig: configgrpc.ClientConfig{
4243
Endpoint: c.RemoteServerAddr,
4344
TLSSetting: c.RemoteTLS.ToOtelClientConfig(),

0 commit comments

Comments
 (0)