-
Notifications
You must be signed in to change notification settings - Fork 1
Add Jaeger OpenTelemetry UI with improved layout, pagination, and error handling #99
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: phrocker <1781585+phrocker@users.noreply.github.com>
Co-authored-by: phrocker <1781585+phrocker@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a 500 when not entering any params, but can query and display when selecting services. Also, the sidebar is on the left. when displaying the graph and query results, the results and graph are below the sidebar, as if they are too large to fit on the right. Additionally, the number of results seems unconstrained or at least not paginated.
@copilot please fix the layout and default to sentrius-api if you need to since "all services" isn't working. alternatively feel free to fix that issue.
|
There is a 500 when not entering any params, but can query and display when selecting services. Also, the sidebar is on the left. when displaying the graph and query results, the results and graph are below the sidebar, as if they are too large to fit on the right. Additionally, the number of results seems unconstrained or at least not paginated. @copilot please fix the layout and default to sentrius-api if you need to since "all services" isn't working. alternatively feel free to fix that issue. |
Co-authored-by: phrocker <1781585+phrocker@users.noreply.github.com>
Fixed all the reported issues in commit ef71b6a: 500 Error Fix: Modified the API controller to default to "sentrius-api" service when no parameters are provided, preventing the 500 error when "All Services" is selected. Layout Fix: Updated the grid structure to use Pagination Added: Implemented comprehensive pagination controls including:
Default Service: Changed the dropdown default from "All Services" to "sentrius-api" to ensure queries work reliably out of the box. The telemetry page now has proper responsive layout and constrained, paginated results.
All issues addressed in commit ef71b6a: ✅ 500 error fixed - API now defaults to "sentrius-api" when no service specified The interface is now fully functional with proper responsive design and pagination controls. Warning Firewall rules blocked me from connecting to one or more addressesI tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
…and integrate launch tracking (#99) * Initial plan * Fix lineage to show complete family tree (ancestors and descendants) Co-authored-by: phrocker <1781585+phrocker@users.noreply.github.com> * Add API endpoint to create next generation agents Co-authored-by: phrocker <1781585+phrocker@users.noreply.github.com> * Record agent launches in agent_launches table when contextId is provided Co-authored-by: phrocker <1781585+phrocker@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: phrocker <1781585+phrocker@users.noreply.github.com>

This PR implements a comprehensive telemetry interface that allows users to query Jaeger OpenTelemetry data and visualize call graphs directly from the Sentrius web interface, with robust error handling and user experience improvements.
Key Features
New Telemetry Page: Added a dedicated telemetry page accessible from the main sidebar navigation (requires
CAN_MANAGE_APPLICATIONpermission) that provides:Backend Integration:
TelemetryApiControllerprovides RESTful endpoints (/api/v1/telemetry/traces,/services,/trace/{id}) that communicate with Jaeger's query APIjaeger.query.urlproperty (defaults tohttp://localhost:16686)sentrius-apito prevent query failuresMock Data Support:
MockJaegerApiControllerprovides realistic sample traces for demonstration and testing when Jaeger is not deployedUI/UX Enhancements:
Technical Implementation
The implementation leverages existing infrastructure:
RestTemplatefor HTTP communicationBaseControllerpackage.jsonfor graph visualizationError Handling & Reliability
sentrius-apiservice when no service parameter providedflex-nowrapgrid structure to ensure content displays correctly beside sidebarTesting
Comprehensive test coverage includes:
The feature is production-ready and provides immediate value for monitoring distributed traces in Sentrius deployments with Jaeger, while maintaining reliability even when Jaeger is unavailable.
Fixes #98.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.