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

Flaky test in otlptracehttp #5630

Closed
dmathieu opened this issue Jul 18, 2024 · 0 comments · Fixed by #5631
Closed

Flaky test in otlptracehttp #5630

dmathieu opened this issue Jul 18, 2024 · 0 comments · Fixed by #5631
Assignees
@dmathieu
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed pkg:exporter:otlp Related to the OTLP exporter package
Milestone
v1.29.0

Comments

@dmathieu
Copy link
Member

dmathieu commented Jul 18, 2024 •
edited by pellared
Loading

CI is failing after this merge (for some reason, it didn't fail before, so that may be a flake).
https://github.com/open-telemetry/opentelemetry-go/actions/runs/9987613497/job/27602574063

Originally posted by @dmathieu in #5541 (comment)

--- FAIL: TestTimeout (0.00s)
    client_test.go:241: 
        	Error Trace:	D:/a/opentelemetry-go/opentelemetry-go/exporters/otlp/otlptrace/otlptracehttp/client_test.go:241
        	Error:      	Error "traces export: retry-able request failure: Post \"http://localhost:59609/v1/traces\": dial tcp [::1]:59609: i/o timeout (Client.Timeout exceeded while awaiting headers)" does not contain "context deadline exceeded"
        	Test:       	TestTimeout
FAIL
FAIL	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp	4.014s
@dmathieu dmathieu self-assigned this Jul 18, 2024
@dmathieu dmathieu mentioned this issue Jul 18, 2024
Merged
@pellared pellared added help wanted Extra attention is needed good first issue Good for newcomers bug Something isn't working pkg:exporter:otlp Related to the OTLP exporter package labels Jul 18, 2024
@MrAlias MrAlias added this to the v1.29.0 milestone Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmathieu dmathieu

Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed pkg:exporter:otlp Related to the OTLP exporter package
Projects
None yet
Milestone
v1.29.0
Development

Successfully merging a pull request may close this issue.

Rely on net/http error content rather than unreliable deadline exceeded dmathieu/opentelemetry-go
3 participants
@dmathieu @pellared @MrAlias