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][exporter/elasticsearch] TestExporterLogs tests flaky on Windows #37286

Closed
pjanotti opened this issue Jan 17, 2025 · 4 comments · Fixed by #37288
Closed

[flaky test][exporter/elasticsearch] TestExporterLogs tests flaky on Windows #37286

pjanotti opened this issue Jan 17, 2025 · 4 comments · Fixed by #37288
Labels
exporter/elasticsearch flaky test a test is flaky os:windows

Comments

@pjanotti
Copy link
Contributor

Component(s)

exporter/elasticsearch

Describe the issue you're reporting

https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/12815700710/job/35735010165#step:8:528

=== FAIL: . TestExporterLogs/no_retry/fail_item/retry.enabled_is_false (0.28s)
    exporter_test.go:577: 
        	Error Trace:	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/exporter/elasticsearchexporter/exporter_test.go:577
        	Error:      	Not equal: 
        	            	expected: 1
        	            	actual  : 0
        	Test:       	TestExporterLogs/no_retry/fail_item/retry.enabled_is_false
make[2]: *** [../../Makefile.Common:134: test] Error 1

=== FAIL: . TestExporterLogs/no_retry/fail_item (0.00s)

=== FAIL: . TestExporterLogs/no_retry/fail_http_request/retry.enabled_is_false (0.27s)
    exporter_test.go:577: 
make[1]: *** [Makefile:200: exporter/elasticsearchexporter] Error 2
        	Error Trace:	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/exporter/elasticsearchexporter/exporter_test.go:577
make[1]: *** Waiting for unfinished jobs....
        	Error:      	Not equal: 
        	            	expected: 1
        	            	actual  : 0
        	Test:       	TestExporterLogs/no_retry/fail_http_request/retry.enabled_is_false

=== FAIL: . TestExporterLogs/no_retry/fail_http_request (0.00s)

=== FAIL: . TestExporterLogs/no_retry (0.00s)

=== FAIL: . TestExporterLogs (3.46s)

=== FAIL: . TestExporterLogs/no_retry/fail_item/retry.enabled_is_false (re-run 1) (0.40s)
    exporter_test.go:577: 
        	Error Trace:	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/exporter/elasticsearchexporter/exporter_test.go:577
        	Error:      	Not equal: 
        	            	expected: 1
        	            	actual  : 0
        	Test:       	TestExporterLogs/no_retry/fail_item/retry.enabled_is_false

=== FAIL: . TestExporterLogs/no_retry/fail_item (re-run 1) (0.00s)

=== FAIL: . TestExporterLogs/no_retry (re-run 1) (0.00s)

=== FAIL: . TestExporterLogs (re-run 1) (0.41s)
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners for exporter/elasticsearch: @JaredTan95 @carsonip @lahsivjar. See Adding Labels via Comments if you do not have permissions to add labels yourself. For example, comment '/label priority:p2 -needs-triaged' to set the priority and remove the needs-triaged label.

@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@pjanotti pjanotti added the flaky test a test is flaky label Jan 17, 2025
@carsonip
Copy link
Contributor

Thanks for the report, I'm looking into it.

@carsonip carsonip mentioned this issue Jan 17, 2025
Merged
@carsonip
Copy link
Contributor

PR #37288

codeboten pushed a commit that referenced this issue Jan 17, 2025
@carsonip
[chore][exporter/elasticsearch] Fix flaky TestExporterLogs (#37288)
ce1d15a 
#### Description

Fix flaky TestExporterLogs test

<!-- Issue number (e.g. #1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes #37286

<!--Describe what testing was performed and which tests were added.-->
#### Testing

<!--Describe the documentation added.-->
#### Documentation

<!--Please delete paragraphs that you did not use before submitting.-->
@github-actions github-actions bot mentioned this issue Jan 21, 2025
Open
chengchuanpeng pushed a commit to chengchuanpeng/opentelemetry-collector-contrib that referenced this issue Jan 26, 2025
@carsonip @chengchuanpeng
[chore][exporter/elasticsearch] Fix flaky TestExporterLogs (open-tele…
ff62308 
…metry#37288)

#### Description

Fix flaky TestExporterLogs test

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#37286

<!--Describe what testing was performed and which tests were added.-->
#### Testing

<!--Describe the documentation added.-->
#### Documentation

<!--Please delete paragraphs that you did not use before submitting.-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
exporter/elasticsearch flaky test a test is flaky os:windows
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[chore][exporter/elasticsearch] Fix flaky TestExporterLogs carsonip/opentelemetry-collector-contrib
2 participants
@pjanotti @carsonip