Skip to content
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

Allow not specifying the key for link patterns #746

Closed
mckornfield opened this issue Apr 26, 2021 · 7 comments
Closed

Allow not specifying the key for link patterns #746

mckornfield opened this issue Apr 26, 2021 · 7 comments
Labels

Comments

@mckornfield
Copy link

mckornfield commented Apr 26, 2021

Requirement - what kind of business use case are you trying to solve?

  • Allowing a user to configure a link pattern for all links patterns of a certain type to allow easy filtering, e.g. for all link patterns of type tag, add a quick filter:
{
          "type": "tag",
          "url": "https://my-jaeger.com/search?tags=%7B%#{key}%22%3A%#{value}%22%7D",
          "text": "Filter by: #{key}"
}

Problem - what in Jaeger blocks you from solving the requirement?

The fact that the key field is required for Link Patterns

Proposal - what do you suggest to solve the problem or improve the existing situation?

Allowing the key field to be optional, or maybe a boolean applyForAllOfType: true, that instead replaces the key with a reference to the name key and its value accessible via value

Any open questions to address

@mckornfield mckornfield changed the title Allow not specifying the key for link patterns to apply across all of the same type Allow not specifying the key for link patterns Apr 26, 2021
@yurishkuro
Copy link
Member

yurishkuro commented Apr 26, 2021

Is the search via Jaeger UI by the given tag the intended use case? Is that the only use case? Because then it might be better to hardcode this, e.g. show a magnifying glass icon on hover and leave the link patterns for their original deep linking purposes.

@albertteoh
Copy link
Contributor

cc @yoave23

@mckornfield
Copy link
Author

That's the main usecase @yurishkuro , agreed hard-coding might be a better solution. One other maybe useful thing would be if you wanted to also cross reference with another tag, like a process tag while templating (not sure if that's possible now)

@yoave23
Copy link
Contributor

yoave23 commented Oct 13, 2021

@yurishkuro I believe #817 resolves this

@jkowall
Copy link
Contributor

jkowall commented Nov 16, 2021

@yurishkuro does #817 fix this issue? If so, we can close.

@jkowall jkowall added the bug label Nov 16, 2021
@yurishkuro
Copy link
Member

We should ask the OP, @mckornfield

@mckornfield
Copy link
Author

Yep I think it does, I'll close fixed by #817

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants