Repository navigation
[FEATURE] ExemplarMetadataTooltip: display the links of the exemplar (perses#3445) - #442
Draft
austinlparker wants to merge 3 commits into
Draft
austinlparker wants to merge 3 commits into
austinlparker wants to merge 3 commits into
Conversation
Signed-off-by: Augustin Husson <augustin.husson@amadeus.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: Augustin Husson <augustin.husson@amadeus.com>
…(perses#3445) Exemplars can carry links since perses/spec#118, e.g. to open their trace in a tracing UI. List them in the exemplar tooltip, above the labels so they stay in view with long label sets. The links open in a new tab. Their URLs are built by query plugins from exemplar labels, so links with an unsafe scheme are not rendered. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Signed-off-by: Austin Parker <austin@ap2.io>
4 of 5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Part of perses/perses#3445, step 2 ("Embed links") of the exemplars roadmap.
Exemplars can carry links since perses/spec#118, for example to open their trace in a tracing UI. This lists them in a "Links" section of
ExemplarMetadataTooltip:nameshows its URL.javascript:) is not rendered, using the existingsanitizeUrl.Since the tooltip follows the mouse until it is pinned, the links can only be clicked once the tooltip is pinned.
Depends on #429. The
linksfield only exists from@perses-dev/spec0.4.0-beta.0, so this branch carries the two commits of #429 rebased onmain; only the last commit belongs to this PR. Thetype-checkfailure inplugin-system/src/components/LinksEditor/LinksEditor.tsxcomes from that bump and is also present on #429. I will rebase once #429 is merged.The links are produced by the Prometheus plugin in perses/plugins#927.
Tested with unit tests (links rendered with their attributes, URL fallback, unsafe URL dropped, no section without links) and in a Perses build from
mainusing this branch, with TimeSeriesChart and HeatMapChart panels on a Prometheus fed by the OpenTelemetry Collector spanmetrics connector: the link of a pinned exemplar opens its trace in Jaeger.Screenshots
Tooltip with a named link and a link without a name (the URL is shown):
In a dashboard, with the links produced by perses/plugins#927:
Checklist
[<catalog_entry>] <commit message>naming convention using one of thefollowing
catalog_entryvalues:FEATURE,ENHANCEMENT,BUGFIX,BREAKINGCHANGE,DOC,IGNORE.UI Changes
🤖 Generated with Claude Code