Skip to content

Commit

Permalink
[fix][query] Correct query server legacy condition (#6120)
Browse files Browse the repository at this point in the history
## Which problem is this PR solving?
- Towards #6026 

## Description of the changes
- Fixes an issue from #6055
where the condition for routing to the legacy implementation was
accidentally flipped.

## How was this change tested?
- CI

## 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`

Signed-off-by: Mahad Zaryab <[email protected]>
  • Loading branch information
mahadzaryab1 authored Oct 25, 2024
1 parent 2fbc067 commit 9f4e8f7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmd/query/app/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ func NewServer(
}

var grpcServer *grpc.Server
if separatePorts {
if !separatePorts {
grpcServer, err = createGRPCServerLegacy(ctx, options, tm)
} else {
grpcServer, err = createGRPCServerOTEL(ctx, options, tm, telset)
Expand Down

0 comments on commit 9f4e8f7

Please sign in to comment.