Page MenuHomePhabricator

[qa] change default number of rpc test jobs
ClosedPublic

Authored by schancel on May 31 2018, 02:19.

Details

Summary

As per title. Depends on D1400

Test Plan

./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
update-num-jobs
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2689
Build 3490: Bitcoin ABC Buildbot (legacy)
Build 3489: arc lint + arc unit

Event Timeline

jasonbcox added inline comments.
test/functional/test_runner.py
162

This value will only be 4 with a 12-core+ processor. Is the intention to lower this value for typical 4 to 8 core processors?

jasonbcox requested changes to this revision.Jun 4 2018, 21:33

+1 on the default as we discussed in slack. The // 3 is fine since 1 job doesn't necessarily mean 1 node.

This revision now requires changes to proceed.Jun 4 2018, 21:33
This revision is now accepted and ready to land.Jun 4 2018, 21:34
This revision was automatically updated to reflect the committed changes.