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

Generalize backend opentracing tag #1499

Open
szuecs opened this issue Aug 5, 2020 · 2 comments
Open

Generalize backend opentracing tag #1499

szuecs opened this issue Aug 5, 2020 · 2 comments

Comments

@szuecs
Copy link
Member

szuecs commented Aug 5, 2020

#1495 introduced a way to get opentracing tag skipper.backend_name
This https://github.com/zalando/skipper/pull/1495/files#diff-6174ff750fcb17e93febb4b1ee2aec34R727 would be nice if we could use the PreProcessor for default routes like in https://github.com/zalando/skipper/blob/master/skipper.go#L1179

The challenge is that we don't really always have a name.

@marcinzaremba
Copy link
Contributor

Does it make sense to overall make names for backends first class citizen in Skipper then? For sources that do not provide it (Ingress, direct eskip definition, etc.) that could be autogenerated like route names are (according to some sensible pattern). On the other hand isn't it much, is there any other use case that can be simplified/benefit from backends having names?

@aryszka
Copy link
Contributor

aryszka commented Aug 6, 2020

Maybe but not sure. The route objects can have filters, that can set any key/value combination. Just like how we set the tracingTag. Since the routes are usually generated from a higher level representation like the ingress, we can say that the current way is fine for the majority of the use cases. On the routing level, or when the input is in eskip format, it would make sense only if it was possible to define the backends as definitions separate from the routes, and the routes would only reference them by their name. Then it would be a similar scenario as what we have with the route groups.

Of course there can be other ways and use cases that I am currently missing.

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

No branches or pull requests

3 participants