-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[elasticsearchexporter] support compression level #37260
[elasticsearchexporter] support compression level #37260
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, looking forward to collector support.
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
e7d0e11
to
8c14fd2
Compare
8c14fd2
to
d33b245
Compare
the changelog entry is added but the pipleline does not recognize it |
ae427bc
to
3f51059
Compare
070cd73
to
3a7e565
Compare
can we merge this? The pipeline does not seem to be failing for related changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase / merge main
Description
This PR syncs support for configuring compression level on ES exporter. Compression Level is a supported param on
confighttp
. Ref https://github.com/open-telemetry/opentelemetry-collector/blob/main/config/configcompression/compressiontype.go#L16-L18Testing
Added UT
Documentation
The documentation is also updated with default level in case no level is provided