Skip to content

Commit

Permalink
Revert "[exporter/prometheusremotewriteexporter]chore: log warning ab…
Browse files Browse the repository at this point in the history
…out remote_write_queue.num_consumers being a no-op" (open-telemetry#35845)

Reverts open-telemetry#34993
  • Loading branch information
ArthurSens authored and sbylica-splunk committed Dec 17, 2024
1 parent 8265a32 commit 34a40a1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
2 changes: 1 addition & 1 deletion exporter/prometheusremotewriteexporter/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ The following settings can be optionally configured:
- `remote_write_queue`: fine tuning for queueing and sending of the outgoing remote writes.
- `enabled`: enable the sending queue (default: `true`)
- `queue_size`: number of OTLP metrics that can be queued. Ignored if `enabled` is `false` (default: `10000`)
- `num_consumers`: minimum number of workers to use to fan out the outgoing requests. (default: `5`) **WARNING:** Currently, num_consumers doesn't have any effect due to incompatibility with Prometheus remote write API. The value will be ignored. Please see https://github.com/open-telemetry/opentelemetry-collector/issues/2949 for more information.
- `num_consumers`: minimum number of workers to use to fan out the outgoing requests. (default: `5`)
- `resource_to_telemetry_conversion`
- `enabled` (default = false): If `enabled` is `true`, all the resource attributes will be converted to metric labels by default.
- `target_info`: customize `target_info` metric
Expand Down
4 changes: 0 additions & 4 deletions exporter/prometheusremotewriteexporter/factory.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,6 @@ func createMetricsExporter(ctx context.Context, set exporter.Settings,
return nil, errors.New("invalid configuration")
}

if prwCfg.RemoteWriteQueue.NumConsumers != 0 {
set.Logger.Warn("Currently, remote_write_queue.num_consumers doesn't have any effect due to incompatibility with Prometheus remote write API. The value will be ignored. Please see https://github.com/open-telemetry/opentelemetry-collector/issues/2949 for more information.")
}

prwe, err := newPRWExporter(prwCfg, set)
if err != nil {
return nil, err
Expand Down

0 comments on commit 34a40a1

Please sign in to comment.