Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update CI builder image to use bitcoind 0.16.0 and reduce number of required blocks #1134

Merged
merged 5 commits into from
Mar 1, 2018
Merged

Update CI builder image to use bitcoind 0.16.0 and reduce number of required blocks #1134

merged 5 commits into from
Mar 1, 2018

Commits on Mar 1, 2018

  1. contrib: Updated builder image to include v0.16.0 of bitcoind 

    Now that 0.16.0 is stable we should make use of it

Signed-off-by: Christian Decker <decker.christian@gmail.com>
    @cdecker @rustyrussell
    cdecker authored and rustyrussell committed Mar 1, 2018
    Configuration menu
    Copy the full SHA
    490ecc4 View commit details
    Browse the repository at this point in the history

  2. pytest: Require bitcoin-0.16.0 and reduce number of generated blocks 

    0.16.0 is required since we rely on it for some tests and the block
reduction allows us to waste less time during setup. 121 blocks were
chosen so that we have at least one mature output to spend.

Signed-off-by: Christian Decker <decker.christian@gmail.com>
    @cdecker @rustyrussell
    cdecker authored and rustyrussell committed Mar 1, 2018
    Configuration menu
    Copy the full SHA
    3545dab View commit details
    Browse the repository at this point in the history

  3. test_lightningd.py: Enable bech32 test for DEVELOPER 

    Since bitcoind 0.16 is already released it is safe to enable
this test already.
    @rustyrussell
    Jan Sarenik authored and rustyrussell committed Mar 1, 2018
    Configuration menu
    Copy the full SHA
    05e4370 View commit details
    Browse the repository at this point in the history

  4. test_lightning.py: reduce 121 to 101 blocks, remove segwit comment. 

    FWIW, the tests without valgrind take 662 seconds before we reduced
the number of blocks, and only 648 seconds now.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
    @rustyrussell
    rustyrussell committed Mar 1, 2018
    Configuration menu
    Copy the full SHA
    d444d37 View commit details
    Browse the repository at this point in the history

  5. README.md: no need for segwit activation on regtest any more. 

    Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
    @rustyrussell
    rustyrussell committed Mar 1, 2018
    Configuration menu
    Copy the full SHA
    985362c View commit details
    Browse the repository at this point in the history