Page MenuHomePhabricator

roqqit (Roqqit)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 12 2024, 16:34 (31 w, 2 d)

Recent Activity

Thu, Jan 16

roqqit closed D17538: [test] Make test_getpeerinfo more robust by using mocktime.
Thu, Jan 16, 21:07
roqqit committed rABCd2683db55d84: [test] Make test_getpeerinfo more robust by using mocktime (authored by roqqit).
[test] Make test_getpeerinfo more robust by using mocktime
Thu, Jan 16, 21:07
roqqit updated the diff for D17538: [test] Make test_getpeerinfo more robust by using mocktime.

Remove errant newline

Thu, Jan 16, 18:02
roqqit updated the diff for D17538: [test] Make test_getpeerinfo more robust by using mocktime.

Use assert_equal

Thu, Jan 16, 18:00

Wed, Jan 15

roqqit closed D17531: [test] Fix intermittent timeouts in various functional tests.
Wed, Jan 15, 22:25
roqqit committed rABC18aa9533dcc9: [test] Fix intermittent timeouts in various functional tests (authored by MarcoFalke <*~=`'#}+{/-|&$^_@721217.xyz>).
[test] Fix intermittent timeouts in various functional tests
Wed, Jan 15, 22:25
roqqit committed rABC4b8c65801635: test: Fail if connect_nodes fails (authored by laanwj <126646+laanwj@users.noreply.github.com>).
test: Fail if connect_nodes fails
Wed, Jan 15, 22:25
roqqit closed D17530: test: Fail if connect_nodes fails.
Wed, Jan 15, 22:25
roqqit closed D17539: [test] Cleanup unnecessary sync_all in rpc_net.py.
Wed, Jan 15, 22:24
roqqit committed rABCc801d8fc621a: [test] Cleanup unnecessary sync_all in rpc_net.py (authored by roqqit).
[test] Cleanup unnecessary sync_all in rpc_net.py
Wed, Jan 15, 22:24
roqqit closed D17529: [test] clarify rpc_net & p2p_disconnect_ban functional tests.
Wed, Jan 15, 22:24
roqqit committed rABC7878d95a0db2: [test] clarify rpc_net & p2p_disconnect_ban functional tests (authored by Wladimir J. van der Laan <laanwj@protonmail.com>).
[test] clarify rpc_net & p2p_disconnect_ban functional tests
Wed, Jan 15, 22:24
roqqit requested review of D17539: [test] Cleanup unnecessary sync_all in rpc_net.py.
Wed, Jan 15, 18:23
roqqit updated the diff for D17530: test: Fail if connect_nodes fails.

Fix wait_until_helper -> self.wait_until

Wed, Jan 15, 17:29
roqqit added inline comments to D17530: test: Fail if connect_nodes fails.
Wed, Jan 15, 17:26
roqqit requested review of D17529: [test] clarify rpc_net & p2p_disconnect_ban functional tests.
Wed, Jan 15, 17:20
roqqit requested review of D17538: [test] Make test_getpeerinfo more robust by using mocktime.
Wed, Jan 15, 17:19
roqqit updated the summary of D17530: test: Fail if connect_nodes fails.
Wed, Jan 15, 00:31
roqqit updated the diff for D17530: test: Fail if connect_nodes fails.

Add back accidentally deleted comment

Wed, Jan 15, 00:13
roqqit requested review of D17531: [test] Fix intermittent timeouts in various functional tests.
Wed, Jan 15, 00:11
roqqit requested review of D17530: test: Fail if connect_nodes fails.
Wed, Jan 15, 00:07
roqqit requested review of D17529: [test] clarify rpc_net & p2p_disconnect_ban functional tests.
Wed, Jan 15, 00:01

Mon, Jan 13

roqqit closed D17525: [avalanche] Set vote status for highest ranking contender.
Mon, Jan 13, 21:56
roqqit committed rABC1c0e73e93ef7: [avalanche] Set vote status for highest ranking contender (authored by roqqit).
[avalanche] Set vote status for highest ranking contender
Mon, Jan 13, 21:56
roqqit closed D17524: [avalanche] Simplify lock into one-liner in processor_tests.
Mon, Jan 13, 21:55
roqqit committed rABCbadfc35a942d: [avalanche] Simplify lock into one-liner in processor_tests (authored by roqqit).
[avalanche] Simplify lock into one-liner in processor_tests
Mon, Jan 13, 21:55
roqqit requested review of D17525: [avalanche] Set vote status for highest ranking contender.
Mon, Jan 13, 19:59
roqqit requested review of D17524: [avalanche] Simplify lock into one-liner in processor_tests.
Mon, Jan 13, 19:49

Fri, Jan 10

roqqit closed D17503: [avalanche] Set all local stake winners as accepted contenders.
Fri, Jan 10, 16:46
roqqit committed rABCe9b4cbef3b43: [avalanche] Set all local stake winners as accepted contenders (authored by roqqit).
[avalanche] Set all local stake winners as accepted contenders
Fri, Jan 10, 16:46
roqqit updated the diff for D17503: [avalanche] Set all local stake winners as accepted contenders.

Bump mocktime beyond the max registration time so there is not a bunch of flaky winners in the functional test

Fri, Jan 10, 00:36
roqqit planned changes to D17503: [avalanche] Set all local stake winners as accepted contenders.
Fri, Jan 10, 00:15
roqqit requested review of D17503: [avalanche] Set all local stake winners as accepted contenders.
Fri, Jan 10, 00:02

Thu, Jan 9

roqqit closed D17498: [avalanche] Re-sort pollable contenders by reward rank.
Thu, Jan 9, 21:30
roqqit committed rABC073bfa519c06: [avalanche] Re-sort pollable contenders by reward rank (authored by roqqit).
[avalanche] Re-sort pollable contenders by reward rank
Thu, Jan 9, 21:30
roqqit updated the diff for D17498: [avalanche] Re-sort pollable contenders by reward rank.

Reuse numPollable

Thu, Jan 9, 16:49

Wed, Jan 8

roqqit requested review of D17498: [avalanche] Re-sort pollable contenders by reward rank.
Wed, Jan 8, 22:39
roqqit accepted D17497: [avalanche] Fix UB in the reward rank comparator.
Wed, Jan 8, 21:45
roqqit closed D17495: [avalanche] Refactor reward ranking logic into its own comparator.
Wed, Jan 8, 20:37
roqqit committed rABC11d2e76dd3c3: [avalanche] Refactor reward ranking logic into its own comparator (authored by roqqit).
[avalanche] Refactor reward ranking logic into its own comparator
Wed, Jan 8, 20:37
roqqit updated the diff for D17495: [avalanche] Refactor reward ranking logic into its own comparator.

Fix circular dependency

Wed, Jan 8, 18:25
roqqit requested review of D17495: [avalanche] Refactor reward ranking logic into its own comparator.
Wed, Jan 8, 17:44
roqqit closed D17479: [avalanche] Add a way to get pollable contenders from contender cache.
Wed, Jan 8, 16:53
roqqit committed rABC53d16b29628d: [avalanche] Add a way to get pollable contenders from contender cache (authored by roqqit).
[avalanche] Add a way to get pollable contenders from contender cache
Wed, Jan 8, 16:53

Tue, Jan 7

roqqit updated the diff for D17479: [avalanche] Add a way to get pollable contenders from contender cache.

Cleanup unnecessary nesting when sorting contenders and make the sorting work the same as staking rewards selection.

Tue, Jan 7, 22:22
roqqit updated the diff for D17479: [avalanche] Add a way to get pollable contenders from contender cache.
  • Fix rare sorting case where two proofs have the same rank.
  • Make num pollable a parameter so the processor will have control over how many contenders to poll for.
  • Return number of contenders instead of bool.
  • Fix test not actually checking the contender order.
Tue, Jan 7, 20:04

Mon, Jan 6

roqqit updated the diff for D17479: [avalanche] Add a way to get pollable contenders from contender cache.

Move static_assert to processor.h in order to avoid circular dependency

Mon, Jan 6, 23:36
roqqit requested review of D17479: [avalanche] Add a way to get pollable contenders from contender cache.
Mon, Jan 6, 22:57
roqqit closed D17475: [avalanche] Make ComputeProofRewardRank const.
Mon, Jan 6, 21:49
roqqit committed rABC576d79f82e1a: [avalanche] Make ComputeProofRewardRank const (authored by roqqit).
[avalanche] Make ComputeProofRewardRank const
Mon, Jan 6, 21:49
roqqit requested review of D17475: [avalanche] Make ComputeProofRewardRank const.
Mon, Jan 6, 18:54

Sat, Dec 21

roqqit accepted D17382: test: Avoid intermittent block download timeout in p2p_ibd_stalling.
Sat, Dec 21, 02:33
roqqit closed D17381: test: Avoid race after connect_nodes.
Sat, Dec 21, 02:30
roqqit committed rABCefbf025838b6: test: Avoid race after connect_nodes (authored by MarcoFalke <falke.marco@gmail.com>).
test: Avoid race after connect_nodes
Sat, Dec 21, 02:30

Fri, Dec 20

roqqit requested review of D17381: test: Avoid race after connect_nodes.
Fri, Dec 20, 21:32

Thu, Dec 19

roqqit closed D17373: Fix flakiness in tests that wait for peer info to update immediately after disconnecting peers.
Thu, Dec 19, 17:26
roqqit committed rABC7f213483007a: Fix flakiness in tests that wait for peer info to update immediately after… (authored by roqqit).
Fix flakiness in tests that wait for peer info to update immediately after…
Thu, Dec 19, 17:26
roqqit requested review of D17373: Fix flakiness in tests that wait for peer info to update immediately after disconnecting peers.
Thu, Dec 19, 16:43

Dec 18 2024

roqqit closed D17337: [avalanche] Set local stake winner in the contender cache.
Dec 18 2024, 22:45
roqqit committed rABCb1cea59f8291: [avalanche] Set local stake winner in the contender cache (authored by roqqit).
[avalanche] Set local stake winner in the contender cache
Dec 18 2024, 22:45
roqqit closed D17368: Fix race to check chaintips in p2p_headers_sync_with_minchainwork.
Dec 18 2024, 22:37
roqqit committed rABC607be5e6570a: Fix race to check chaintips in p2p_headers_sync_with_minchainwork (authored by roqqit).
Fix race to check chaintips in p2p_headers_sync_with_minchainwork
Dec 18 2024, 22:37
roqqit closed D17345: [avalanche] Make sure each proof is added to contender cache before mining the next block in contender voting test.
Dec 18 2024, 22:37
roqqit committed rABC0657e8a86eeb: [avalanche] Make sure each proof is added to contender cache before mining the… (authored by roqqit).
[avalanche] Make sure each proof is added to contender cache before mining the…
Dec 18 2024, 22:37
roqqit requested review of D17337: [avalanche] Set local stake winner in the contender cache.
Dec 18 2024, 17:26
roqqit updated the diff for D17345: [avalanche] Make sure each proof is added to contender cache before mining the next block in contender voting test.
  • just use getStakeContenderStatus instead of a new accessor
  • hasstakecontender -> getstakecontendervote per feedback
  • pass in prevblockhash and proofid instead of stakecontenderid
Dec 18 2024, 17:23
roqqit updated the diff for D17368: Fix race to check chaintips in p2p_headers_sync_with_minchainwork.

feedback

Dec 18 2024, 16:50

Dec 17 2024

roqqit requested review of D17368: Fix race to check chaintips in p2p_headers_sync_with_minchainwork.
Dec 17 2024, 22:35
roqqit closed D17349: [seeder] Bump thread stacksize.
Dec 17 2024, 21:02
roqqit committed rABC61b79c87cb21: [seeder] Bump thread stacksize (authored by roqqit).
[seeder] Bump thread stacksize
Dec 17 2024, 21:02
roqqit retitled D17349: [seeder] Bump thread stacksize from [seeder] Respect resource limit for stacksize as set by the system to [seeder] Bump thread stacksize.
Dec 17 2024, 20:38
roqqit updated the diff for D17349: [seeder] Bump thread stacksize.

Just bump the value because predicting it is proving to be difficult and time consuming

Dec 17 2024, 20:37

Dec 16 2024

roqqit requested review of D17337: [avalanche] Set local stake winner in the contender cache.
Dec 16 2024, 18:20
roqqit updated the diff for D17345: [avalanche] Make sure each proof is added to contender cache before mining the next block in contender voting test.

Use a debug RPC. Waiting for getavalanchepeerinfo is not sufficient here because its possible for this value to be updated before the contender is added to the cache.

Dec 16 2024, 17:49

Dec 13 2024

roqqit closed D17348: [seeder] Return an error when thread creation fails.
Dec 13 2024, 21:06
roqqit committed rABC200513e58adb: [seeder] Return an error when thread creation fails (authored by roqqit).
[seeder] Return an error when thread creation fails
Dec 13 2024, 21:06
roqqit closed D17350: [seeder] Reword outdated note in readme.
Dec 13 2024, 21:05
roqqit committed rABC82d1ab6e576c: [seeder] Reword outdated note in readme (authored by roqqit).
[seeder] Reword outdated note in readme
Dec 13 2024, 21:05
roqqit requested review of D17350: [seeder] Reword outdated note in readme.
Dec 13 2024, 20:43
roqqit requested review of D17348: [seeder] Return an error when thread creation fails.
Dec 13 2024, 20:27
roqqit requested review of D17349: [seeder] Bump thread stacksize.
Dec 13 2024, 20:27
roqqit updated the test plan for D17349: [seeder] Bump thread stacksize.
Dec 13 2024, 20:26
roqqit planned changes to D17348: [seeder] Return an error when thread creation fails.
Dec 13 2024, 20:08
roqqit planned changes to D17349: [seeder] Bump thread stacksize.
Dec 13 2024, 20:08
roqqit requested review of D17349: [seeder] Bump thread stacksize.
Dec 13 2024, 20:08
roqqit requested review of D17348: [seeder] Return an error when thread creation fails.
Dec 13 2024, 19:37

Dec 12 2024

roqqit requested review of D17345: [avalanche] Make sure each proof is added to contender cache before mining the next block in contender voting test.
Dec 12 2024, 22:08
roqqit accepted D17343: [CI] Fix a type in the build configuration file.
Dec 12 2024, 20:36
roqqit accepted D17341: [CI] Don't run the electrum tests in parallel of the functional tests.
Dec 12 2024, 20:18
roqqit closed D17336: [avalanche] Restrict scope of cs_stakingRewards lock during cleanup.
Dec 12 2024, 17:34
roqqit committed rABC44da17657e7c: [avalanche] Restrict scope of cs_stakingRewards lock during cleanup (authored by roqqit).
[avalanche] Restrict scope of cs_stakingRewards lock during cleanup
Dec 12 2024, 17:34

Dec 11 2024

roqqit requested review of D17337: [avalanche] Set local stake winner in the contender cache.
Dec 11 2024, 23:43
roqqit requested review of D17336: [avalanche] Restrict scope of cs_stakingRewards lock during cleanup.
Dec 11 2024, 17:48

Dec 10 2024

roqqit closed D17331: Add symlink to most recent functional test failure directory.
Dec 10 2024, 19:57
roqqit committed rABCe9d7a6220c01: Add symlink to most recent functional test failure directory (authored by roqqit).
Add symlink to most recent functional test failure directory
Dec 10 2024, 19:57
roqqit closed D17330: Run more functional tests in parallel by default.
Dec 10 2024, 18:08
roqqit committed rABC8917a6f8af6c: Run more functional tests in parallel by default (authored by roqqit).
Run more functional tests in parallel by default
Dec 10 2024, 18:08
roqqit updated the test plan for D17331: Add symlink to most recent functional test failure directory.
Dec 10 2024, 17:36
roqqit updated the diff for D17331: Add symlink to most recent functional test failure directory.
  • move lastfailure to top of build_dir
  • ignore OSError exceptions on windows (didn't use contextlib.suppress because I don't think we should ignore it on Linux)
Dec 10 2024, 17:36
roqqit planned changes to D17331: Add symlink to most recent functional test failure directory.
Dec 10 2024, 16:51