Page MenuHomePhabricator
Differential D2155

[qa] Switch to using multiprocessing instead of threading
AbandonedPublic
Actions

Authored by schancel on Nov 30 2018, 03:20.

Details

Reviewers
deadalnix
jasonbcox
Group Reviewers
Restricted Project
Summary

This commit changes the test_runner to use the multiprocessing library
rather than the threading library. This enables the capturing STDIN and STDOUT
since the queues now use IPC rather than python internal threading.

Depends on D2153, D2168

Test Plan
./test/functional/test_runner.py

Diff Detail

Event Timeline

schancel created this revision.Nov 30 2018, 03:20
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 30 2018, 03:20
Herald added a reviewer: deadalnix. · View Herald Transcript
teamcity added a subscriber: teamcity.Nov 30 2018, 03:21

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

schancel added a child revision: D2156: [qa] Dyanmically import tests so we can start adding decorators.Nov 30 2018, 03:24
teamcity added a comment.Nov 30 2018, 03:29

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4184: Diff 6189.Nov 30 2018, 03:29
schancel added a comment.Nov 30 2018, 04:27
This comment was removed by schancel.
schancel edited the summary of this revision. (Show Details)Nov 30 2018, 05:34
schancel edited the summary of this revision. (Show Details)Nov 30 2018, 21:57
schancel added a parent revision: D2153: [qa] Make src_dir handling more robust.
schancel removed a parent revision: D2154: [qa] Fix portseed calculation to avoid collisions.
schancel updated this revision to Diff 6223.Dec 1 2018, 21:33

Split diff so move is handled before this. DEpends on D2168

schancel edited the summary of this revision. (Show Details)Dec 1 2018, 21:33
schancel added a parent revision: D2168: [qa] [move only] Move threading helpers to be top level functions..
teamcity added a comment.Dec 1 2018, 21:35

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

teamcity added a comment.Dec 1 2018, 21:43

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing failed.

Harbormaster failed remote builds in B4218: Diff 6223!Dec 1 2018, 21:43
teamcity added a comment.Dec 5 2018, 04:53

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

teamcity added a comment.Dec 5 2018, 05:02

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing failed.

jasonbcox requested changes to this revision.Feb 27 2019, 20:35
jasonbcox added a subscriber: jasonbcox.

Moving this off of the review queue until the dependencies are finished.

This revision now requires changes to proceed.Feb 27 2019, 20:35
schancel abandoned this revision.Mar 17 2019, 04:33

Revision Contents
Changeset List

PathSize
test/
functional/
60 lines

Diff 6223

View Options

test/functional/test_runner.py

Loading...