Skip to content

Commit

Permalink
Remove sending_queue from AWS Prometheus Remote Write Exporter (#3186)
Browse files Browse the repository at this point in the history
* Remove sending_queue from AWS Prometheus Remote Write Exporter

This is a follow up of open-telemetry/opentelemetry-collector#2951.

Fixes #3163.

* Fix the test
  • Loading branch information
rakyll authored Apr 21, 2021
1 parent ab68513 commit 45fb933
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 14 deletions.
12 changes: 2 additions & 10 deletions exporter/awsprometheusremotewriteexporter/config_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,11 +60,6 @@ func TestLoadConfig(t *testing.T) {
TypeVal: "awsprometheusremotewrite",
},
TimeoutSettings: exporterhelper.DefaultTimeoutSettings(),
QueueSettings: exporterhelper.QueueSettings{
Enabled: true,
NumConsumers: 2,
QueueSize: 10,
},
RetrySettings: exporterhelper.RetrySettings{
Enabled: true,
InitialInterval: 10 * time.Second,
Expand All @@ -81,12 +76,9 @@ func TestLoadConfig(t *testing.T) {
},
Insecure: false,
},
ReadBufferSize: 0,

ReadBufferSize: 0,
WriteBufferSize: 512 * 1024,

Timeout: 5 * time.Second,

Timeout: 5 * time.Second,
Headers: map[string]string{
"prometheus-remote-write-version": "0.1.0",
"x-scope-orgid": "234"},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,6 @@ exporters:
awsprometheusremotewrite:
awsprometheusremotewrite/2:
namespace: "test-space"
sending_queue:
enabled: true
num_consumers: 2
queue_size: 10
retry_on_failure:
enabled: true
initial_interval: 10s
Expand Down

0 comments on commit 45fb933

Please sign in to comment.