Skip to content

[elasticsearchexporter] support setting query parameters on outgoing bulk request#44480

Merged
ChrsMark merged 16 commits into
open-telemetry:mainfrom
khushijain21:parameters
Dec 17, 2025
Merged

[elasticsearchexporter] support setting query parameters on outgoing bulk request#44480
ChrsMark merged 16 commits into
open-telemetry:mainfrom
khushijain21:parameters

Conversation

@khushijain21
Copy link
Copy Markdown
Contributor

@khushijain21 khushijain21 commented Nov 24, 2025

Description

This PR adds support for parsing query parameters from the endpoint

Testing

Added UT

Documentation

None

@khushijain21 khushijain21 changed the title support setting query parameters on outgoing bulk request [elasticsearchexporter] support setting query parameters on outgoing bulk request Nov 24, 2025
carsonip pushed a commit to elastic/go-docappender that referenced this pull request Nov 25, 2025
This PR is part of effort to support ouput.parameters option available on beats to beatreceivers

Related Issue where more explanation is provided: elastic/beats#47764
Related PR on ES exporter that should follow after this: open-telemetry/opentelemetry-collector-contrib#44480
@khushijain21 khushijain21 marked this pull request as ready for review November 25, 2025 11:44
@ChrsMark
Copy link
Copy Markdown
Member

@carsonip @JaredTan95 want to take a look? Otherwise I guess this can be merged.

Copy link
Copy Markdown
Contributor

@carsonip carsonip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

Comment thread exporter/elasticsearchexporter/bulkindexer.go Outdated
Comment thread exporter/elasticsearchexporter/bulkindexer.go Outdated
@ChrsMark ChrsMark merged commit 73768b4 into open-telemetry:main Dec 17, 2025
189 checks passed
@github-actions github-actions Bot added this to the next release milestone Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants