When using HTTP reporters in the clients to export tracing data directly to the collectors, users still want to be able to use remote sampler that loads centrally managed sampling strategies.
The /sampling endpoint is only implemented in the agent today.
Expose the same endpoint from the collectors, e.g. /api/sampling.
When this is done, update FAQ docs, cf. https://github.com/jaegertracing/documentation/pull/335
Hi
If this is still open, I can take it up.
It's yours.
Hi @namratachaudhary I have already submitted a PR for this issue.
Open question: can clients be configured with a different sampling URL?
Clients support the following config options:
http(s)://collector:port/api/tracesPossible solutions:
/sampling or /baggage path added by the client as needed.http(s)://collector:port/api used by both span reporter (by adding /traces) and sampler (by adding /sampling).Q: is it better to hardcode the final segments of the URL in the clients or make them configurable? Strictly speaking the URL path is part of the contract, e.g. if we introduce a different format of the sampling output it should become /api/v2/sampling or something like that.
cc @jpkrohling @pavolloffay @objectiser
Q: is it better to hardcode the final segments of the URL in the clients or make them configurable?
I would lean towards the consistency and require configuring the full URL. It's more flexible and one could deploy it on different path.
@pavolloffay so you'd vote for JAEGER_SAMPLING_ENDPOINT?
This is done.