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
Branch
teamcity_migrate_ibd
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7341
Build 12725: Bitcoin ABC Buildbot (legacy)
Build 12724: arc lint + arc unit

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

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

26

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

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

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.