Patchwork D8112: test: pin the number of CPU for issue4074 tests

login
register
mail settings
Submitter phabricator
Date Feb. 13, 2020, 12:45 a.m.
Message ID <differential-rev-PHID-DREV-dg2qet2m6uwiambvntee-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/45194/
State Superseded
Headers show

Comments

phabricator - Feb. 13, 2020, 12:45 a.m.
marmoute created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION 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.

REPOSITORY
  rHG Mercurial

BRANCH
  stable

REVISION DETAIL
  https://phab.mercurial-scm.org/D8112

AFFECTED FILES
  tests/test-issue4074.t

CHANGE DETAILS




To: marmoute, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/tests/test-issue4074.t b/tests/test-issue4074.t
--- a/tests/test-issue4074.t
+++ b/tests/test-issue4074.t
@@ -25,5 +25,5 @@ 
 
 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)