Skip to content

elasticsearchexporter: add http.response.status_code to logs#45829

Merged
edmocosta merged 11 commits into
open-telemetry:mainfrom
marclop:f/add-status.code-to-failed-log-messages
Mar 4, 2026
Merged

elasticsearchexporter: add http.response.status_code to logs#45829
edmocosta merged 11 commits into
open-telemetry:mainfrom
marclop:f/add-status.code-to-failed-log-messages

Conversation

@marclop

@marclop marclop commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Description

Expose HTTP status on failed document logs to allow for better filtering and error analysis.

Testing

Unit tested

Expose HTTP status on failed document logs to allow for better filtering
and error analysis.

Signed-off-by: Marc Lopez Rubio <marc5.12@outlook.com>
Signed-off-by: Marc Lopez Rubio <marc5.12@outlook.com>
Signed-off-by: Marc Lopez Rubio <marc5.12@outlook.com>
Signed-off-by: Marc Lopez Rubio <marc5.12@outlook.com>
@marclop marclop marked this pull request as ready for review February 10, 2026 03:38
@marclop marclop requested a review from a team as a code owner February 10, 2026 03:38
@marclop marclop requested a review from crobert-1 February 10, 2026 03:38

@carsonip carsonip left a comment

Copy link
Copy Markdown
Contributor

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
@github-actions

Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Feb 25, 2026
@marclop marclop changed the title elasticsearchexporter: add status.code to logs elasticsearchexporter: add http.response.status_code to logs Mar 2, 2026

@carsonip carsonip left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

code change lgtm. Stale changelog

Comment thread .chloggen/f_add-status.code-to-failed-log-messages.yaml Outdated
Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
@github-actions github-actions Bot removed the Stale label Mar 3, 2026
@edmocosta edmocosta merged commit 1624260 into open-telemetry:main Mar 4, 2026
191 checks passed
@otelbot

otelbot Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution @marclop! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. If you are getting started contributing, you can also join the CNCF Slack channel #opentelemetry-new-contributors to ask for guidance and get help.

antonio-mazzini pushed a commit to antonio-mazzini/opentelemetry-collector-contrib that referenced this pull request Mar 5, 2026
…telemetry#45829)

#### Description

Expose HTTP status on failed document logs to allow for better filtering
and error analysis.

#### Testing
Unit tested

---------

Signed-off-by: Marc Lopez Rubio <marc5.12@outlook.com>
Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
avleentwilio pushed a commit to avleentwilio/opentelemetry-collector-contrib that referenced this pull request Apr 1, 2026
…telemetry#45829)

#### Description

Expose HTTP status on failed document logs to allow for better filtering
and error analysis.

#### Testing
Unit tested

---------

Signed-off-by: Marc Lopez Rubio <marc5.12@outlook.com>
Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
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.

5 participants