Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[exporters/otlptracehttp] Got error messages despite export succeeded #3706

Open
songy23 opened this issue Feb 10, 2023 · 4 comments · May be fixed by #3707
Open

[exporters/otlptracehttp] Got error messages despite export succeeded #3706

songy23 opened this issue Feb 10, 2023 · 4 comments · May be fixed by #3707
Assignees
@songy23
Labels
bug Something isn't working

Comments

@songy23
Copy link
Member

songy23 commented Feb 10, 2023

Description

When using the otlptracehttp exporter to export trace to a custom endpoint, I got error messages despite export succeeded:

2023/02/10 10:34:31 traces export: failed to send to [my own trace agent]: 202 Accepted

/cc @dineshg13

Environment

  • OS: MacOS
  • Architecture: Apple M1
  • Go Version: 1.19
  • opentelemetry-go version: v1.13.0

Steps To Reproduce

  1. Have a mock backend that returns 202 upon successfully receiving OTLP HTTP trace payloads
  2. Run otlptracehttp exporter against the mock backend
  3. See error despite traces are successfully exported

Expected behavior

No error logs upon 202 Accepted responses for otlptracehttp exporter.
@songy23 songy23 added the bug Something isn't working label Feb 10, 2023
@MrAlias
Copy link
Contributor

MrAlias commented Feb 10, 2023

Please include a full run-able example of the error. Including the response your mock server is returning.

@MrAlias MrAlias added the response needed Waiting on user input before progress can be made label Feb 10, 2023
@songy23 songy23 linked a pull request Feb 10, 2023 that will close this issue
Open
@songy23
Copy link
Member Author

songy23 commented Feb 10, 2023

Hi @MrAlias I have a unit test to reproduce it and a fix in #3707. Please assign this issue to me.

@MrAlias MrAlias removed the response needed Waiting on user input before progress can be made label Feb 17, 2023
@tomjo
Copy link

tomjo commented Jun 16, 2023

We're also encountering this issue when sending traces to new-relic

@songy23
Copy link
Member Author

songy23 commented Jun 16, 2023

This is currently blocked by the spec: open-telemetry/opentelemetry-specification#3203

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@songy23 songy23

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[exporters/otlptracehttp] Do not log errors for 2XX response songy23/opentelemetry-go
3 participants
@tomjo @MrAlias @songy23