Implement GRPC Client For Interacting With Remote Storage #6789
Assignees
Labels
Comments
4 tasks
github-merge-queue bot
pushed a commit
that referenced
this issue
Mar 5, 2025
## Which problem is this PR solving? - Towards #6789 ## Description of the changes - This PR moves the internal `TracesData` type to package `jptrace` so that it can be used by the client for GRPC storage as well. ## 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`: `npm run lint` and `npm run test` --------- Signed-off-by: Mahad Zaryab <[email protected]>
Merged
4 tasks
github-merge-queue bot
pushed a commit
that referenced
this issue
Mar 9, 2025
…dependency_storage.proto` (#6819) <!-- !! Please DELETE this comment before posting. We appreciate your contribution to the Jaeger project! 👋🎉 --> ## Which problem is this PR solving? - Towards #6789 ## Description of the changes - This PR removes the gogoproto annotations from `trace_storage.proto` and `dependency_storage.proto` to avoid introducing a dependency on gogo for users. Instead, it uses a similar patching mechanism to api_v3 to add the annotations when generating the *.pb.go files. ## 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`: `npm run lint` and `npm run test` --------- Signed-off-by: Mahad Zaryab <[email protected]>
This was referenced Mar 9, 2025
github-merge-queue bot
pushed a commit
that referenced
this issue
Mar 9, 2025
## Which problem is this PR solving? - Towards #6789 ## Description of the changes - This PR addresses #6829 (comment) and changes the remote storage api v2 to return a chunk of traces instead of a single trace from `GetTraces` and `FindTraces` ## 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`: `npm run lint` and `npm run test` --------- Signed-off-by: Mahad Zaryab <[email protected]>
github-merge-queue bot
pushed a commit
that referenced
this issue
Mar 11, 2025
…rage api v2 (#6829) <!-- !! Please DELETE this comment before posting. We appreciate your contribution to the Jaeger project! 👋🎉 --> ## Which problem is this PR solving? - Towards #6789 ## Description of the changes - This PR implements a `TraceReader` for the remote storage API v2 along with `GetServices` call. ## How was this change tested? - Added a unit test ## 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`: `npm run lint` and `npm run test` --------- Signed-off-by: Mahad Zaryab <[email protected]>
4 tasks
github-merge-queue bot
pushed a commit
that referenced
this issue
Mar 15, 2025
…torage api v2 (#6843) <!-- !! Please DELETE this comment before posting. We appreciate your contribution to the Jaeger project! 👋🎉 --> ## Which problem is this PR solving? - Towards #6789 ## Description of the changes - This PR implements the `GetOperations` call in the gRPC tracereader ## How was this change tested? - Added a unit test ## 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`: `npm run lint` and `npm run test` --------- Signed-off-by: Mahad Zaryab <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As part of #6629, we defined the API for the v2 remote storage API. We now want to implement a client that will interact with that API.
The text was updated successfully, but these errors were encountered: