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

Fix fail count calculation #3723

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

dougkei
Copy link

@dougkei dougkei commented Dec 23, 2019

lastFetch is in millisecond while the divisor is interval in second

lastFetch is in millisecond while the divisor is interval in second
@pivotal-issuemaster
Copy link

@dougkei Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@dougkei Thank you for signing the Contributor License Agreement!

@codecov
Copy link

codecov bot commented Dec 23, 2019

Codecov Report

Merging #3723 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3723   +/-   ##
=========================================
  Coverage     65.84%   65.84%           
  Complexity     1593     1593           
=========================================
  Files           203      203           
  Lines          7455     7455           
  Branches        877      877           
=========================================
  Hits           4909     4909           
  Misses         2225     2225           
  Partials        321      321
Impacted Files Coverage Δ Complexity Δ
...rk/cloud/netflix/eureka/EurekaHealthIndicator.java 15.78% <0%> (ø) 2 <0> (ø) ⬇️

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 02f3bb8...0e5d33e. Read the comment docs.

@spencergibb
Copy link
Member

Can you describe the error you are seeing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants