From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 2C75D1F953; Wed, 27 Oct 2021 21:15:15 +0000 (UTC) Date: Wed, 27 Oct 2021 21:15:15 +0000 From: Eric Wong To: Thomas =?utf-8?Q?Wei=C3=9Fschuh?= Cc: meta@public-inbox.org Subject: lei-q-remote-import failures [was: [PATCH] t/index-git-times: support non-master default branch] Message-ID: <20211027211515.M84746@dcvr> References: <20211025222453.252181-1-thomas@t-8ch.de> <20211025225822.GA28875@dcvr> <20211026052815.GA13265@dcvr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: List-Id: Thomas Weißschuh wrote: > On 2021-10-26 05:28+0000, Eric Wong wrote: > > Thomas Weißschuh wrote: > > > The only failures I still see are in t/lei-q-remote-import.t which is unrelated > > > to the git branch and some more that fail only during parallel test execution. > > > > > > In t/lei-q-remote-import.t it seems the search results are not memoized to the > > > local store. (See attachment) > > > > Thanks, I'll check out lei-q-remote-import.t separately. > > That could be related to SMP or slow/fast storage. > > Any details you can share about CPU core count, speeds > > or storage speeds, or CPU scheduler, CONFIG_HZ? > > CPU: > > Model name: 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz > CPU family: 6 > Model: 140 > Thread(s) per core: 2 > Core(s) per socket: 4 > Socket(s): 1 > Stepping: 1 > CPU max MHz: 4700,0000 > CPU min MHz: 400,0000 > > Storage: Samsung EVO 970 (with plenty of memory cache) > CONFIG_HZ: 300 I'm puzzled by this failure; I'm not sure if I've ever seen it. Does it fail when run standalone? (prove -bvw t/lei-q-remote-import.t) Maybe running under TEST_LEI_ERR_LOUD=1 or checking syslog can reveal something. Adding a sleep shouldn't be necessary, but maybe something else is broken I'm not seeing...: diff --git a/t/lei-q-remote-import.t b/t/lei-q-remote-import.t index 92d8c9b6058c..a2d643cd06c7 100644 --- a/t/lei-q-remote-import.t +++ b/t/lei-q-remote-import.t @@ -35,6 +35,7 @@ test_lei({ tmpdir => $tmpdir }, sub { lei_ok(@cmd, '-I', $url); is_deeply($slurp_emls->($o), $exp1, 'got results after remote search'); unlink $o or BAIL_OUT $!; + sleep 1; lei_ok(@cmd); ok(-f $o && -s _, 'output exists after import but is not empty') or diag $lei_err;