Skip to content

Conversation

@iblancasa
Copy link
Contributor

Description

Link to tracking issue

Fixes #43950

Testing

Tested with config:

exporters:
  loadbalancing:
    retry_on_failure:
      enabled: true
      initial_interval: 5s
      max_interval: 30s
      max_elapsed_time: 60s
    sending_queue:
      enabled: true
      num_consumers: 10
      queue_size: 10000
      block_on_overflow: false
      sizer: items
    protocol:
      otlp:
        timeout: 1s
        tls:
          insecure: true
    resolver:
      dns:
        hostname: otel-receivers.test
        port: '4317'
        interval: 5s
        timeout: 1s

@l00py
Copy link
Contributor

l00py commented Nov 4, 2025

@iblancasa -- awesome quick turnaround time. Tested it and working as you expected.
One thing that I'm unsure is that the metrics will report twice, at loadbalancing and otlp. Any thoughts on that? :)
I can imagine this being valid in case where the end-user could use both the loadbalancing/otlp and otlp exporters. Seems like the loadbalancing/otlp will also be counted in otlp as well.

Here's an example:

image

loadbalancing

otelcol_exporter_sent_log_records_total{env="test",exporter="loadbalancing",exporter_host_name="otel-exporter",host_name="otel-exporter",instance="41ac214b-c822-4677-aa9d-f588e730dfdb",job="otel-exporter",server_port="8888",service_instance_id="41ac214b-c822-4677-aa9d-f588e730dfdb",service_name="otel-exporter",service_version="0.138.0-dev",team="observability",url_scheme="http",workload="exporter"}

otlp

otelcol_exporter_sent_log_records_total{env="test",exporter="otlp",exporter_host_name="otel-exporter",host_name="otel-exporter",instance="41ac214b-c822-4677-aa9d-f588e730dfdb",job="otel-exporter",server_port="8888",service_instance_id="41ac214b-c822-4677-aa9d-f588e730dfdb",service_name="otel-exporter",service_version="0.138.0-dev",team="observability",url_scheme="http",workload="exporter"}

@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Nov 6, 2025
@songy23 songy23 merged commit 5aa2746 into open-telemetry:main Nov 6, 2025
205 of 206 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 6, 2025
jelly-afk pushed a commit to jelly-afk/opentelemetry-collector-contrib that referenced this pull request Nov 6, 2025
…ation succeeds (open-telemetry#43960)

#### Description

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#43950

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Tested with config:
```yaml
exporters:
  loadbalancing:
    retry_on_failure:
      enabled: true
      initial_interval: 5s
      max_interval: 30s
      max_elapsed_time: 60s
    sending_queue:
      enabled: true
      num_consumers: 10
      queue_size: 10000
      block_on_overflow: false
      sizer: items
    protocol:
      otlp:
        timeout: 1s
        tls:
          insecure: true
    resolver:
      dns:
        hostname: otel-receivers.test
        port: '4317'
        interval: 5s
        timeout: 1s
```

Signed-off-by: Israel Blancas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exporter/loadbalancing ready to merge Code review completed; ready to merge by maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[exporter/loadbalancing] Bug introduced in #43721 - Fails to create exporter per endpoints

6 participants