User Details
User Details
- User Since
- Jun 12 2024, 16:34 (46 w, 3 d)
Thu, May 1
Thu, May 1
roqqit abandoned D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
see D18036
I see
roqqit committed rABC029da727864b: [avalanche] Fix incomplete testing of contenders when staking rewards are active (authored by Fabien).
[avalanche] Fix incomplete testing of contenders when staking rewards are active
teamcity awarded D18036: [avalanche] Fix incomplete testing of contenders when staking rewards are active a Doubloon token.
roqqit updated the diff for D18036: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
cleanup remaining issues from old draft
Wed, Apr 30
Wed, Apr 30
roqqit added inline comments to D18028: Properly check the node polls for contenders upon restart with no prior block being mined.
roqqit committed rABC2568103e82b9: [avalanche] Fix dangling proof flakiness in contender test (authored by roqqit).
[avalanche] Fix dangling proof flakiness in contender test
roqqit updated the summary of D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
roqqit added a comment to D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
see updated summary. skipping the end of the test is not ideal but it is unclear how much investigation that last piece will take. on the other hand, fixing the flakiness in the rest of the test ensures we are not regressing on the majority of contenders logic. CI deserves to be green again.
roqqit updated the summary of D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
teamcity awarded D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active a Doubloon token.
roqqit added a comment to D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
@bot build-master
roqqit updated the diff for D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
fix various flakiness issues
Tue, Apr 29
Tue, Apr 29
roqqit requested review of D18022: [avalanche] Fix dangling proof flakiness in contender test.
roqqit committed rABC8ebbda579e21: [avalanche] Only re-add contenders to the cache if the remote proof is present (authored by roqqit).
[avalanche] Only re-add contenders to the cache if the remote proof is present
roqqit committed rABC4a35b7c33298: [avalanche] Correctly check if a contender is associated with a remote proof (authored by roqqit).
[avalanche] Correctly check if a contender is associated with a remote proof
roqqit updated the diff for D18011: [avalanche] Correctly check if a contender is associated with a remote proof.
address feedback
roqqit added inline comments to D18011: [avalanche] Correctly check if a contender is associated with a remote proof.
roqqit committed rABC209972a1e872: [test] Call out missing messages when asserting the debug log (authored by roqqit).
[test] Call out missing messages when asserting the debug log
roqqit committed rABC5d39d807f0a9: [avalanche] Also log when contenders are not promoted (authored by roqqit).
[avalanche] Also log when contenders are not promoted
roqqit committed rABC58a4750550e8: [avalanche] Rename isRemoteProof to hasRemoteProofStatus (authored by roqqit).
[avalanche] Rename isRemoteProof to hasRemoteProofStatus
Mon, Apr 28
Mon, Apr 28
roqqit requested review of D18012: [avalanche] Also log when contenders are not promoted.
roqqit requested review of D18010: [avalanche] Rename isRemoteProof to hasRemoteProofStatus.
roqqit committed rABC7f65b6a8f20e: [test] Pretty print expected messages when asserting a debug log (authored by roqqit).
[test] Pretty print expected messages when asserting a debug log
roqqit committed rABC645747705d1f: [test] Swap debug log and expected messages error string (authored by roqqit).
[test] Swap debug log and expected messages error string
roqqit added inline comments to D18009: [test] Call out missing messages when asserting the debug log.
roqqit requested changes to D18004: [avalanche] Delay polling the conflicting transactions.
roqqit updated the diff for D18009: [test] Call out missing messages when asserting the debug log.
pretty print missing messages too
roqqit updated the diff for D18009: [test] Call out missing messages when asserting the debug log.
rebase
roqqit updated the diff for D18008: [test] Pretty print expected messages when asserting a debug log.
rebase
roqqit updated the diff for D18007: [test] Swap debug log and expected messages error string.
It turns out I did break them
roqqit requested review of D18009: [test] Call out missing messages when asserting the debug log.
roqqit added a comment to D18007: [test] Swap debug log and expected messages error string.
Both failures appear to be I2P related. I don't think it was this diff that triggered the failures.
roqqit requested review of D18008: [test] Pretty print expected messages when asserting a debug log.
roqqit requested review of D18007: [test] Swap debug log and expected messages error string.
Sun, Apr 27
Sun, Apr 27
teamcity awarded D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active a Piece of Eight token.
roqqit added a comment to D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
@bot build-master
roqqit updated the test plan for D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
teamcity awarded D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active a Haypence token.
roqqit added a comment to D18003: [avalanche] Fix incomplete testing of contenders when staking rewards are active.
@bot build-master
Sat, Apr 26
Sat, Apr 26
roqqit committed rABC8ed7061ec857: [avalanche] Fix staking reward flakiness after node restart in contenders test (authored by roqqit).
[avalanche] Fix staking reward flakiness after node restart in contenders test
Fri, Apr 25
Fri, Apr 25
roqqit committed rABCe11cfb3ca8eb: [avalanche] Promote and poll contenders as soon as quorum is active (authored by roqqit).
[avalanche] Promote and poll contenders as soon as quorum is active
Thu, Apr 24
Thu, Apr 24
roqqit updated the diff for D17986: [avalanche] Promote and poll contenders as soon as quorum is active.
feedback
roqqit committed rABCda39027ba95c: [avalanche] Refactor proof rank calculation in functional test (authored by roqqit).
[avalanche] Refactor proof rank calculation in functional test
roqqit requested review of D17993: [avalanche] Refactor proof rank calculation in functional test.
Wed, Apr 23
Wed, Apr 23
roqqit requested review of D17986: [avalanche] Promote and poll contenders as soon as quorum is active.
roqqit committed rABCec671e1e6767: [avalanche] Prevent regressions in the case that promotion happens twice for a… (authored by roqqit).
[avalanche] Prevent regressions in the case that promotion happens twice for a…
roqqit committed rABC79f659739aab: [doc] Add productivity note about utilizing the ramdisk for faster testing (authored by roqqit).
[doc] Add productivity note about utilizing the ramdisk for faster testing
Tue, Apr 22
Tue, Apr 22
roqqit requested review of D17972: [doc] Add productivity note about utilizing the ramdisk for faster testing.
Fri, Apr 11
Fri, Apr 11
roqqit requested changes to D17832: [avalanche] Poll for the transactions that didn't make it into the last block.
roqqit committed rABCc50432fcd1a8: [avalanche] Wait for the local winner instead of immediately asserting (authored by roqqit).
[avalanche] Wait for the local winner instead of immediately asserting
Thu, Apr 10
Thu, Apr 10
roqqit requested review of D17923: [avalanche] Wait for the local winner instead of immediately asserting.
roqqit committed rABCe0776c77fd38: [avalanche] Remove requirement that local staking rewards have to be computed… (authored by roqqit).
[avalanche] Remove requirement that local staking rewards have to be computed…
Wed, Apr 9
Wed, Apr 9
Add Shibusawa activation logic
Tue, Apr 8
Tue, Apr 8
roqqit requested review of D17912: Add Shibusawa activation logic.