[CI] Increase stream buffer limit for build subprocess
ClosedPublic
Actions

Authored by jasonbcox on Jun 26 2020, 23:55.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCc87b967f347e: [CI] Increase stream buffer limit for build subprocess

Summary

This fixes the error currently seen in secp256k1 tests: https://build.bitcoinabc.org/viewLog.html?buildId=88041&buildTypeId=BitcoinABC_Master_BitcoinAbcMasterSecp256k1&tab=buildLog&_focus=256

Test Plan

Test secp256k1 in CI with this patch: https://build.bitcoinabc.org/viewLog.html?buildId=88107&buildTypeId=BitcoinABC_BitcoinAbcStaging&tab=buildResultsDiv

Diff Detail

Repository

rABC Bitcoin ABC

Branch

buffer-limit

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 11742
Build 23035: Default Diff Build & Tests	Link to the build
Build 23034: Build without Wallet	Link to the build
Build 23033: Build with clang-10	Link to the build
Build 23032: arc lint + arc unit

Event Timeline

jasonbcox created this revision.Jun 26 2020, 23:55

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 26 2020, 23:55

jasonbcox requested review of this revision.Jun 26 2020, 23:55

jasonbcox planned changes to this revision.

jasonbcox requested review of this revision.Jun 27 2020, 00:09

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

Harbormaster completed remote builds in B11742: Diff 21814.Jun 27 2020, 00:19

More context: the bash traces are echoing a list of the valid targets, as it checks for validity when a list is passed to build_with_cmake. This list has no line break and causes the overflow.

This revision is now accepted and ready to land.Jun 27 2020, 06:48

Closed by commit rABCc87b967f347e: [CI] Increase stream buffer limit for build subprocess (authored by jasonbcox). · Explain WhyJun 27 2020, 15:18

This revision was automatically updated to reflect the committed changes.

jasonbcox added a commit: rABCc87b967f347e: [CI] Increase stream buffer limit for build subprocess.