flow-go icon indicating copy to clipboard operation
flow-go copied to clipboard

Debugging Flaky Epoch Tests

Open jordanschalm opened this issue 2 years ago • 1 comments

@durkmurder noticed that periodic halts to approval production were causing flakiness in the static epoch test. Applying the same change here, to disable requiring approvals, to see if it will impact test flakiness on the other test cases.

jordanschalm avatar Aug 03 '22 19:08 jordanschalm

Codecov Report

Merging #2921 (5504951) into master (49f6bf5) will decrease coverage by 0.00%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2921      +/-   ##
==========================================
- Coverage   57.07%   57.06%   -0.01%     
==========================================
  Files         693      693              
  Lines       63087    63087              
==========================================
- Hits        36005    36001       -4     
- Misses      24122    24124       +2     
- Partials     2960     2962       +2     
Flag Coverage Δ
unittests 57.06% <ø> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
module/mempool/epochs/transactions.go 90.32% <0.00%> (-9.68%) :arrow_down:
...s/hotstuff/votecollector/staking_vote_processor.go 83.87% <0.00%> (-3.23%) :arrow_down:
fvm/handler/contract.go 88.59% <0.00%> (ø)
engine/consensus/compliance/core.go 76.51% <0.00%> (ø)
consensus/hotstuff/eventloop/event_loop.go 74.82% <0.00%> (+1.36%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Aug 03 '22 20:08 codecov-commenter

Closing, further investigation on https://github.com/onflow/flow-go/pull/3097

jordanschalm avatar Aug 25 '22 23:08 jordanschalm