This is an archive of the discontinued Mercurial Phabricator instance.

Differential D8112

test: pin the number of CPU for issue4074 tests
ClosedPublic

Authored by marmoute on Feb 12 2020, 7:44 PM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHG60bc043d7df7: test: pin the number of CPU for issue4074 tests
Summary

On machine with an hundreds of CPUs, the "user" CPU time reported can be
inflated by the status steps. Since the test especially focus on the diff
computation, we restrict the number of CPU to avoid potential issues.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Feb 12 2020, 7:44 PM
marmoute added a commit: rHG60bc043d7df7: test: pin the number of CPU for issue4074 tests.Feb 13 2020, 12:56 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHG60bc043d7df7: test: pin the number of CPU for issue4074 tests (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/test-issue4074.t (2 lines)

Diff 20190

tests/test-issue4074.t

$ hg ci -qAm0 $ hg ci -qAm0
Modify it: Modify it:
$ "$PYTHON" ../s.py > a $ "$PYTHON" ../s.py > a
Time a check-in, should never take more than 10 seconds user time: Time a check-in, should never take more than 10 seconds user time:
$ hg ci --time -m1 $ hg ci --time -m1 --config worker.enabled=no
time: real .* secs .user [0-9][.].* sys .* (re) time: real .* secs .user [0-9][.].* sys .* (re)