open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
2.99k stars 2.31k forks source link

Refactor Zipkin Receiver to include Protocols in Config #35730

Open EOjeah opened 3 hours ago

EOjeah commented 3 hours ago

Component(s)

receiver/zipkin

Is your feature request related to a problem? Please describe.

Feature Request Refactor zipkin receiver to include protocols in configuration Current

zipkin:

After

zipkin:
  protocols:
    http:

Describe the solution you'd like

In order to prepare the zipkin receiver for potentially adding a udp receiver as specified in #35620 and reduce the amount of changes associated with one issue. This issue is to refactor the code to accept zipkin configuration with potentially different protocols. Initially accepted is the current http protocol with this change. This will match more closely with the configuration in the otlp receiver that accepts both http and grpc Before

zipkin:

After

zipkin:
  protocols:
    http:

Follow up changes to include udp protocol in another change.

Describe alternatives you've considered

No response

Additional context

Related to #35620

github-actions[bot] commented 3 hours ago

Pinging code owners: