[extension/jaegerremotesampling] Misleading implementation on two endpoints

[yurishkuro]

Component(s)

extension/jaegerremotesampling

What happened?

Description

This extension serves the same output on two HTTP endpoints / and /sampling, which presumably was meant to emulate the Jaeger Agent's behavior. However, in the Jaeger Agent the / endpoint returns a legacy format (see https://www.jaegertracing.io/docs/1.41/apis/#remote-sampling-configuration-stable).

This difference either needs to be called out in the README or fixed (although returning legacy format from this extension does not make sense, I'd rather it did not support this endpoint).

Collector version

0.70.0

Environment information

Environment

any

OpenTelemetry Collector configuration

No response

Log output

No response

Additional context

No response

[github-actions]

Pinging code owners:

• extension/jaegerremotesampling: @jpkrohling @frzifus

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[jpkrohling]

@frzifus, are you able to handle this?

[frzifus]

Sure, i will have a more detailed view next week.

[yurishkuro]

🎉