Page MenuHomePhabricator

Remove magnetic anomaly transitional code, activate on block height
ClosedPublic

Authored by Fabien on Jan 15 2019, 19:52.

Details

Summary

This include the -magneticanomalyactivationtime parameter and some
functional tests.

Depends on D2324

Test Plan
make check
./test/functional/test_runner.py

Run IBD with checkpoint disabled and assumevalid=0 on mainnet and testnet

Diff Detail

Repository
rABC Bitcoin ABC
Branch
magnetic_remove_mtp
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4542
Build 7147: Bitcoin ABC Buildbot (legacy)
Build 7146: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Jan 15 2019, 20:53

You should also remove the mempool clearing code when we reorg accross the fork.

src/consensus/activation.h
20 ↗(On Diff #6676)

int32_t

src/consensus/tx_verify.cpp
41 ↗(On Diff #6676)

Why ?

test/functional/abc-replay-protection.py
110 ↗(On Diff #6676)

It's a good idea to do these in their own patches. It's easy wins that won't get blocked by anything else.

test/functional/abc-transaction-ordering.py
20 ↗(On Diff #6676)

dito

This revision now requires changes to proceed.Jan 15 2019, 20:53

Split commit (python tests are updated in D2324)
int => int32_t
Update transaction unit tests instead of removing them

@deadalnix Didn't you already removed the mempool clearing code in D2213 ? Or are you talking about something else ?

deadalnix requested changes to this revision.Jan 17 2019, 01:00
In D2318#52425, @Fabien wrote:

@deadalnix Didn't you already removed the mempool clearing code in D2213 ? Or are you talking about something else ?

Absolutely, my bad. You are all set.

src/chainparams.cpp
303
This revision now requires changes to proceed.Jan 17 2019, 01:00

The test plan for this change should definitively include an IBD with checkpoint disabled and assumevalid=0 for both mainnet and testnet.

Fabien edited the test plan for this revision. (Show Details)

Fix fork block height on testnet

deadalnix requested changes to this revision.Jan 17 2019, 17:03

Please do run these IBD on mainnet and testnet.

This revision now requires changes to proceed.Jan 17 2019, 17:03

IBD on testnet is OK, still running on mainet.
I will request for review when it's done

Fabien requested review of this revision.Jan 18 2019, 10:38

IBD on both testnet and mainnet were successful with the following parameters:
-checkpoints=0 -assumevalid=0

Fix testnet block height

It seems tome that both height are off by one.

Fabien planned changes to this revision.Jan 19 2019, 17:53

Fix block height for mainnet and testnet

This revision is now accepted and ready to land.Jan 22 2019, 15:06
This revision was automatically updated to reflect the committed changes.