Page MenuHomePhabricator

Updated ibd.sh script to take bitcoind options and then pass them directly to the bitcoind call.
ClosedPublic

Authored by nakihito on Feb 26 2019, 19:04.

Details

Summary

Updated IBD script to take bitcoind options as parameters and pass them directly to the bitcoind call. Also changed the bitcoind call to not run in the background so the script terminates when finished. Completes T524.

Test Plan

ibd.sh -assumevalid=0 -checkpoints=0 -disablewallet
Check debug.log and you should find:

2019-02-27T00:16:09Z Validating signatures for all blocks.
2019-02-27T00:16:14Z Wallet disabled!

There should be no log message for checkpoints.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a reviewer: Restricted Owners Package.Feb 26 2019, 19:04
nakihito removed a subscriber: schancel.
jasonbcox requested changes to this revision.Feb 26 2019, 19:17
jasonbcox added inline comments.
contrib/teamcity/ibd.sh
25 ↗(On Diff #7486)

Any reason $* can't be used in the place of $PARAMS entirely so this line can be removed?

27 ↗(On Diff #7486)

removing & doesn't match what this diff is intending to do.
and it doesn't appear to achieve anything. Please undo.

This revision now requires changes to proceed.Feb 26 2019, 19:17

Readded background option for bitcoind call and removed PARAMS variable.

jasonbcox requested changes to this revision.Feb 26 2019, 20:26

Testplan should indicate that the params passed in were verified in logs or otherwise

This revision now requires changes to proceed.Feb 26 2019, 20:26

I will update the TeamCity config to the following once this is landed:

  1. Existing IBD will have -disablewallet added to ensure the config behavior remains the same
  2. A new IBD setting will be added with -disablewallet -assumevalid=0 -checkpoints=0
This revision is now accepted and ready to land.Mar 1 2019, 22:42
This revision was automatically updated to reflect the committed changes.