Differential D16471

Make PeerManager own a FastRandomContext
ClosedPublic
Actions

Authored by Fabien on Jul 15 2024, 20:13.

Details

Reviewers

roqqit

Group Reviewers

Restricted Project

Commits

rABCe308d2e67b4b: Make PeerManager own a FastRandomContext

Summary

This lets us avoid some non-determinism in tests

Backport of core#28558.

Depends on D16470.

Test Plan

ninja all check check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR28558

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 29565
Build 58665: Build Diff	lint-circular-dependencies · build-without-wallet · build-fuzzer · build-clang-tidy · build-clang · build-diff · build-debug
Build 58664: arc lint + arc unit

Event Timeline

Fabien created this revision.Jul 15 2024, 20:13

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 15 2024, 20:13

Fabien requested review of this revision.Jul 15 2024, 20:13

teamcity edited the summary of this revision. (Show Details)Jul 15 2024, 20:13

Harbormaster completed remote builds in B29565: Diff 48637.Jul 15 2024, 20:32

roqqit accepted this revision.Jul 15 2024, 22:29

This revision is now accepted and ready to land.Jul 15 2024, 22:29

Closed by commit rABCe308d2e67b4b: Make PeerManager own a FastRandomContext (authored by dergoegge <n.goeggi@gmail.com>, committed by Fabien). · Explain WhyJul 16 2024, 07:59

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCe308d2e67b4b: Make PeerManager own a FastRandomContext.

Revision Contents
Changeset List

Path

Size

src/

net_processing.h

3 lines

net_processing.cpp

44 lines

policy/

fees.h

5 lines

fees.cpp

6 lines

test/

fuzz/

fees.cpp

3 lines

policy_fee_tests.cpp

3 lines

util/

setup_common.cpp

1 line

Diff 48637

View Options

src/net_processing.h

View Options

src/net_processing.cpp

View Options

src/policy/fees.h

View Options

src/policy/fees.cpp

View Options

src/test/fuzz/fees.cpp

View Options

src/test/policy_fee_tests.cpp

View Options

src/test/util/setup_common.cpp

Make PeerManager own a FastRandomContextClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 48637

src/net_processing.h

src/net_processing.cpp

src/policy/fees.h

src/policy/fees.cpp

src/test/fuzz/fees.cpp

src/test/policy_fee_tests.cpp

src/test/util/setup_common.cpp

Make PeerManager own a FastRandomContext
ClosedPublic
Actions

Revision Contents
Changeset List