Page MenuHomePhabricator

Add check-buildbot CI config
ClosedPublic

Authored by jasonbcox on Sep 4 2020, 17:23.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCb4c3e530f73a: Add check-buildbot CI config
Summary

So we can run buildbot tests in CI

Depends on D7364

Test Plan

Install dependencies:

cd contrib/teamcity
workon abcbot
pip3 install teamcity-messages deepmerge pytest
pip3 install -r ../buildbot/requirements.txt

Run:

./build-configurations.py check-buildbot

Diff Detail

Repository
rABC Bitcoin ABC
Branch
buildbot-test
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12650
Build 25401: Build Diffbuild-without-wallet · build-diff · build-clang-10 · build-clang-tidy
Build 25400: arc lint + arc unit

Event Timeline

Fabien requested changes to this revision.Sep 4 2020, 20:15
Fabien added a subscriber: Fabien.

A few nits, and needs rebase to use the YAML config

contrib/teamcity/builds/check-buildbot.sh
12 ↗(On Diff #23299)

You don't need the -v option, that will bloat the output with success info.

contrib/teamcity/setup-debian-buster.sh
129 ↗(On Diff #23299)

Weird comment, did you mean Install Python dependencies for the build bot ?

This revision now requires changes to proceed.Sep 4 2020, 20:15
contrib/teamcity/setup-debian-buster.sh
129 ↗(On Diff #23299)

I meant it to be a section where each subproject's python dependencies are installed, but since buildbot is the only one, I can see how this looks weird.

  • Rebase on yml config
  • Use pytest without -v option
  • Fix comment
This revision is now accepted and ready to land.Sep 7 2020, 06:35

Fix buildbot requirements path

This revision was automatically updated to reflect the committed changes.