Page MenuHomePhabricator

[CI] Make IBD a standard build configuration
ClosedPublic

Authored by Fabien on Sep 6 2019, 12:40.

Details

Summary

This makes the CI IBD test a single entry point which can then be
reconfigured.

Test Plan
ABC_BUILD_NAME=build-ibd ./contrib/teamcity/build-configurations.sh
ABC_BUILD_NAME=build-ibd-no-assumevalid-checkpoint \
./contrib/teamcity/build-configurations.sh

Diff Detail

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

Event Timeline

jasonbcox requested changes to this revision.Sep 6 2019, 15:39
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
contrib/teamcity/ibd.sh
18 ↗(On Diff #11121)

any reason to leave the /ibd out of the double quotes? DEBUG_LOG is this way, for example.

26 ↗(On Diff #11121)

This entire block can be one line:
: "${BITCOIND_BINARY:="${TOPLEVEL}/build/src/bitcoind"}"

This revision now requires changes to proceed.Sep 6 2019, 15:39
jasonbcox requested changes to this revision.Sep 6 2019, 17:22
jasonbcox added inline comments.
contrib/teamcity/ibd.sh
26 ↗(On Diff #11121)

Note the double quotes after the :=. This is important incase TOPLEVEL contains spaces.

This revision now requires changes to proceed.Sep 6 2019, 17:22
Fabien requested review of this revision.Sep 6 2019, 17:27
Fabien added inline comments.
contrib/teamcity/ibd.sh
26 ↗(On Diff #11121)

It's OK as the whole assignment is enclosed in double quotes.

This revision is now accepted and ready to land.Sep 6 2019, 20:48

I created a new configuration for calling resource intensive configs via build-configurations. Running build-ibd-no-assumevalid-checkpoint now for sanity.

2 remarks:

  • The IBD failed due to timeout after 1h, which I think is a good value for all the other tests but not for IBD
  • The set -x should be removed during the loop, otherwise it prints a sleep every 100ms. I will update D3992.
jasonbcox requested changes to this revision.Sep 11 2019, 17:09

The changes to ibd.sh got incorporated into D4003. Please rebase and use BITCOIND (or rename it to BITCOIND_BINARY, which is fine too).

This revision now requires changes to proceed.Sep 11 2019, 17:09
This revision is now accepted and ready to land.Sep 12 2019, 15:19
This revision was automatically updated to reflect the committed changes.