Skip to content

Conversation

@AritraDey-Dev
Copy link

Which problem is this PR solving?

Description of the changes

  • Adds the missing backend endpoint to support the Deep Dependencies graph in the UI.

How was this change tested?

  • Ran the server: go run ./cmd/jaeger
  • Ran Jaeger UI: npm start
  • Opened http://localhost:5173/deep-dependencies?service=test and confirmed the graph renders.
image

Checklist

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds backend support for the Deep Dependencies graph feature in the Jaeger UI by exposing a new HTTP endpoint that the frontend can consume.

  • Adds /analytics/v1/dependencies endpoint that routes to the existing deepDependencies handler
  • Applies proper middleware (OpenTelemetry route tagging and span naming) to the new endpoint
  • Includes an integration test to verify the endpoint is accessible and returns the expected response structure

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
cmd/query/app/http_handler.go Registers new /analytics/v1/dependencies route with proper middleware, reusing the existing deepDependencies handler
cmd/all-in-one/all_in_one_test.go Adds integration test checkDeepDependencies to verify the new endpoint is accessible and returns valid JSON structure

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create dummy /deep-dependencies endpoint

1 participant