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

Increase max wait time on host metrics test #1510

Merged

Conversation

james-bebbington
Copy link
Member

The Windows CI sees this test fail very occasionally, presumably because system metric collection is slow (lots of processes running or sys calls blocked for some reason). Try increasing the max wait time before resorting to anything more drastic.

Resolves: #1445 (comment)

@codecov
Copy link

codecov bot commented Aug 6, 2020

Codecov Report

Merging #1510 into master will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1510      +/-   ##
==========================================
+ Coverage   91.27%   91.32%   +0.04%     
==========================================
  Files         240      240              
  Lines       16719    16719              
==========================================
+ Hits        15261    15269       +8     
+ Misses       1047     1043       -4     
+ Partials      411      407       -4     
Impacted Files Coverage Δ
receiver/otlpreceiver/otlp.go 94.44% <0.00%> (+2.77%) ⬆️
translator/internaldata/resource_to_oc.go 91.66% <0.00%> (+6.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 284ea11...7855d9e. Read the comment docs.

@james-bebbington
Copy link
Member Author

james-bebbington commented Aug 6, 2020

Not sure why build-and-test is showing up up twice in the build Checks. The build is succeeding but the earlier (transient) failure won't go away

@bogdandrutu bogdandrutu merged commit 62b80a3 into open-telemetry:master Aug 6, 2020
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug report for failed CircleCI build (job: windows-test)
2 participants