Skip to content

metrics: add metric for when commit is nil#4283

Merged
melekes merged 4 commits intomasterfrom
marko/4263_followup
Jan 7, 2020
Merged

metrics: add metric for when commit is nil#4283
melekes merged 4 commits intomasterfrom
marko/4263_followup

Conversation

@tac0turtle
Copy link
Contributor

Signed-off-by: Marko Baricevic marbar3778@yahoo.com

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

- follow up to #4263
- when a commit is nil, then it should be counted as a missed commit

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
@tac0turtle tac0turtle added the T:observability Type: Observability label Jan 6, 2020
@tac0turtle tac0turtle self-assigned this Jan 6, 2020
Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better to add ForBlock() func (BlockIDFlag == BlockIDFlagCommit)

@codecov-io
Copy link

Codecov Report

Merging #4283 into master will increase coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4283      +/-   ##
==========================================
+ Coverage   65.47%   65.56%   +0.09%     
==========================================
  Files         225      225              
  Lines       19433    19422      -11     
==========================================
+ Hits        12724    12735      +11     
+ Misses       5726     5707      -19     
+ Partials      983      980       -3
Impacted Files Coverage Δ
consensus/state.go 77.14% <100%> (-0.21%) ⬇️
crypto/sr25519/pubkey.go 32.14% <0%> (-7.15%) ⬇️
behaviour/reporter.go 45.45% <0%> (-3.04%) ⬇️
consensus/reactor.go 78.08% <0%> (-0.24%) ⬇️
blockchain/v0/pool.go 79.16% <0%> (+0.32%) ⬆️
consensus/replay.go 72.54% <0%> (+0.78%) ⬆️
p2p/pex/pex_reactor.go 84.18% <0%> (+1.12%) ⬆️
libs/clist/clist.go 68.18% <0%> (+1.51%) ⬆️
libs/events/events.go 98.05% <0%> (+4.85%) ⬆️
... and 1 more

@melekes melekes merged commit d7f4ce3 into master Jan 7, 2020
@melekes melekes deleted the marko/4263_followup branch January 7, 2020 08:09
@leoluk leoluk mentioned this pull request Jan 7, 2020
tac0turtle added a commit that referenced this pull request Jan 8, 2020
- follow up to #4263
- when a commit is nil, then it should be counted as a missed commit

Signed-off-by: Marko Baricevic marbar3778@yahoo.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T:observability Type: Observability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants