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.

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

Flaky test - prometheusreceiver - TestOpenMetricsNegative #9119

Closed
jpkrohling opened this issue Apr 8, 2022 · 9 comments
Closed

Flaky test - prometheusreceiver - TestOpenMetricsNegative #9119

jpkrohling opened this issue Apr 8, 2022 · 9 comments
Labels
bug Something isn't working comp:prometheus Prometheus related issues flaky test a test is flaky receiver/prometheus Prometheus receiver

Comments

@jpkrohling
Copy link
Member

Seen here: https://github.com/open-telemetry/opentelemetry-collector-contrib/runs/5869686660?check_suite_focus=true#logs

2022-04-07T14:06:18.1333771Z === RUN   TestOpenMetricsNegative/bad_type_1
2022-04-07T14:06:18.1334163Z race: limit on 8128 simultaneously alive goroutines is exceeded, dying
2022-04-07T14:06:18.1334861Z FAIL	github.com/open-telemetry/opentelemetry-collector-contrib/receiver/prometheusreceiver	102.534s

cc @Aneurysm9 @dashpole

@jpkrohling jpkrohling added bug Something isn't working flaky test a test is flaky labels Apr 8, 2022
@jpkrohling
Copy link
Member Author

@jpkrohling jpkrohling added the comp:prometheus Prometheus related issues label Apr 8, 2022
@gouthamve
Copy link
Member

Looks like it will be fixed in the next release of Go: golang/go#47056 (comment)

@dmitryax
Copy link
Member

@tigrannajaryan
Copy link
Member

@tigrannajaryan
Copy link
Member

4 failures in 4 days. I think we need to disable the test until it is fixed.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 8, 2022

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

@github-actions github-actions bot added the Stale label Nov 8, 2022
@jpkrohling jpkrohling added receiver/prometheus Prometheus receiver and removed Stale labels Nov 24, 2022
@github-actions
Copy link
Contributor

Pinging code owners: @Aneurysm9 @dashpole. See Adding Labels via Comments if you do not have permissions to add labels yourself.

@jpkrohling
Copy link
Member Author

It looks like this might have been fixed, but a positive confirmation from the code owner(s) is desirable.

@dashpole
Copy link
Contributor

This has been reenabled for a few months without complaint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working comp:prometheus Prometheus related issues flaky test a test is flaky receiver/prometheus Prometheus receiver
Projects
None yet
Development

No branches or pull requests

5 participants