Skip to content

[jaeger] Add new "publicAddress" option #246

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Dec 24, 2021
Merged

Conversation

ricoberger
Copy link
Member

@ricoberger ricoberger commented Dec 24, 2021

The new "publicAddress" option in the Jaeger plugin configuration, can
be used to define the address of a Jaeger UI instance, which can be
accessed by the users. If this option is set, we add a link to each
trace, which allows the users to open the trace in the Jaeger UI.

We are now saving the state of all spans, if they are expanded or not.
This ways we can also render an expanded span, when the user scrolled
the span out of the viewport and when he later scrolls back to the
expanded span.

Before this change, the span was not expanded anymore, when the user
scrolls through the list of spans.

The new "publicAddress" option in the Jaeger plugin configuration, can
be used to define the address of a Jaeger UI instance, which can be
accessed by the users. If this option is set, we add a link to each
trace, which allows the users to open the trace in the Jaeger UI.
We are now saving the state of all spans, if they are expanded or not.
This ways we can also render an expanded span, when the user scrolled
the span out of the viewport and when he later scrolls back to the
expanded span.

Before this change, the span was not expanded anymore, when the user
scrolls through the list of spans.
@ricoberger ricoberger merged commit c6031fa into main Dec 24, 2021
@ricoberger ricoberger deleted the jaeger-public-address branch December 24, 2021 09:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant