Skip to content

Add error details to OTLP GRPC exporter error messages..#5143

Merged
aabmass merged 9 commits into
open-telemetry:mainfrom
DylanRussell:fix_err_detail
May 22, 2026
Merged

Add error details to OTLP GRPC exporter error messages..#5143
aabmass merged 9 commits into
open-telemetry:mainfrom
DylanRussell:fix_err_detail

Conversation

@DylanRussell

@DylanRussell DylanRussell commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Description

Include GRPC error details in the logs.. Right now the GRPC error details field isn't logged anywhere, it is often populated with more details about why the error occured..

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Unit tests

Does This PR Require a Contrib Repo Change?

  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@MikeGoldsmith MikeGoldsmith left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Change to errors look good, but I don't think the uv.lock file are relate - please can you remove it?

@github-project-automation github-project-automation Bot moved this to Approved PRs in Python PR digest Apr 27, 2026
@DylanRussell

Copy link
Copy Markdown
Contributor Author

Woops, removed thanks

Comment thread exporter/opentelemetry-exporter-otlp-proto-grpc/tests/test_otlp_exporter_mixin.py Outdated
@aabmass

aabmass commented May 13, 2026

Copy link
Copy Markdown
Member

@DylanRussell can you update to use new TownCrier based changelog, and fix the merge conflict?

@aabmass aabmass moved this from Approved PRs to Approved PRs that need fixes in Python PR digest May 14, 2026
@DylanRussell

Copy link
Copy Markdown
Contributor Author

I've added the new CHANGELOG format and resolved merge conflicts.. should be good to go

@DylanRussell DylanRussell moved this from Approved PRs that need fixes to Ready for merge in Python PR digest May 21, 2026
@aabmass
aabmass added this pull request to the merge queue May 22, 2026
Merged via the queue into open-telemetry:main with commit 64d0a49 May 22, 2026
479 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for merge to Done in Python PR digest May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants