-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Support for configurable gzip compression_level in elasticsearchexporter #40501
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Pinging code owners for exporter/elasticsearch: @JaredTan95 @carsonip @lahsivjar. See Adding Labels via Comments if you do not have permissions to add labels yourself. For example, comment '/label priority:p2 -needs-triaged' to set the priority and remove the needs-triaged label. |
That setting should be exposed already. See exporter settingscom/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/elasticsearchexporter#http-settings and detailed configurations for http clients |
Indeed, the functionality appears in the documentation, but after testing with a collector-contrib V0.117.0, unfortunately it doesn't work because it doesn't recognise this setting: '/label priority:p2 -waiting-for-author' |
@JhonMartins99 the correct config key is try following: exporters:
elasticsearch/ssg:
compression: gzip
... other fields |
@VihasMakwana in documentation booth are needed for select level of compression: I used both because i need better compression (9) than default compression (1) I test this config: exporters:
elasticsearch/ssg:
endpoints:
- https://my-endpoint:9200
logs_index: myIndex-log
user: xxxxx
password: xxxxx
timeout: 50s
tls:
insecure: false
insecure_skip_verify: true
sending_queue:
enabled: true
num_consumers: 15
queue_size: 5000
compression: gzip
compression_level: 9 |
Oh ok. As you're on 0.117.0, the compression level wasn't supported back then. Take a look at compression settings for that verion. You might wanna update to latest collector version and try out following:
|
@VihasMakwana thankyou very match!! I test it on a 0.118.0 version and it works. ;) |
Component(s)
cmd/otelcontribcol
Is your feature request related to a problem? Please describe.
Feature Request
I’m using the
elasticsearchexporter
in OpenTelemetry Collector version 0.117.0 and would like to configure the gzip compression level used when sending data to Elasticsearch.Currently, the exporter supports enabling gzip compression via
compression: gzip
, but does not expose acompression_level
option in the YAML configuration.Since the exporter uses
confighttp
, andconfighttp
supportscompression_level
, it would be great to expose this setting in the exporter configuration.Use Case
I want to optimize bandwidth usage by increasing the compression level to
9
(gzip.BestCompression), even if it means slightly higher CPU usage.Describe the solution you'd like
Proposed Solution
Add support for a
compression_level
field in the exporter configuration, similar to other exporters that useconfighttp
.Thanks!
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: