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: AS3215 2.0.0.0/16 X-Spam-Status: No, score=-3.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS, UNPARSEABLE_RELAY shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from nautica.notk.org (ipv6.notk.org [IPv6:2001:41d0:1:7a93::1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id AA4831F9FC for ; Wed, 8 Dec 2021 10:56:49 +0000 (UTC) Received: by nautica.notk.org (Postfix, from userid 108) id 9F8E7C01A; Wed, 8 Dec 2021 11:56:45 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codewreck.org; s=2; t=1638961005; bh=wrjZy19NN2/HuVR1lIAcixYeav/KG9jRHS425+Ku0+Q=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=yrD2JFPamttGr9yb6vbmgxIBB0d8M6dOELRDB8OkJekfG0kSm9WRC6sIbmYC+21V2 4+oZN2ckM0QcfI/QZv7YvpN1H/1McHSGPJuTjVifEezPSYRtiBumbR/an2o9nNExKx T+7EyOgg8fzOE6LUy1UZuH3E5kP2PM/bgA366zsY4Mfc2dSX1Ui2xmSBahF7PF0DGb FiUumwb++XunWiR+1WtnhuzXVLc3pCuaP6yZ6sCivtSOixr2oUGUk23Fx+nnzyLHPX tqFn8yUfrQWgpqPHAhi7K6pjJyuu1+2l7gTURqHuhb0eEk2qM5MPYIQSPJqmd7uAVK HMwa2mnSkHyAg== Received: from odin.codewreck.org (localhost [127.0.0.1]) by nautica.notk.org (Postfix) with ESMTPS id 3F13EC009; Wed, 8 Dec 2021 11:56:42 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codewreck.org; s=2; t=1638961004; bh=wrjZy19NN2/HuVR1lIAcixYeav/KG9jRHS425+Ku0+Q=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=tvaFCfMipzypsDfiZALNYBfzCRErRhgO2dTZ3ZKJi8zss9fe1MRs8enxg0nWEV04k 95oRvfkpX4f+wZDyFJx/yUq6pFmhPQrUPkK8bfl4kuQdqgHf5I8dM/qIV0MF7xG1md 6A+lcopFd4UNFzAx1ar7e+WyfbvYiyM3R730JUHBAx+w6ebG20RPeFaKrq9M7XSQGv Uq6zW2mVQGq2vkuxhspLagF62mkDCwb+3Cp6BevjBplyfIvX0EFtU+VCbvhBaX3B/i HVMkELm3RtVVbPfJWAAtfqjz76nhKNzuHnHquKkJn7LSX+gH14U8D7p2Ka+dKjOj7J lg2uL9de//JnQ== Received: from localhost (odin.codewreck.org [local]) by odin.codewreck.org (OpenSMTPD) with ESMTPA id 94054f07; Wed, 8 Dec 2021 10:56:39 +0000 (UTC) Date: Wed, 8 Dec 2021 19:56:24 +0900 From: Dominique Martinet To: Eric Wong Cc: Julien Moutinho , meta@public-inbox.org Subject: Re: Test failures with 1.7.0 Message-ID: References: <20211208010730.f47xxgzj53nwgvja@sourcephile.fr> <20211208040836.GA27368@dcvr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20211208040836.GA27368@dcvr> List-Id: Hi, Eric Wong wrote on Wed, Dec 08, 2021 at 04:08:36AM +0000: > > And another reviewer fails 3 times with: > > > t/extsearch.t (Wstat: 2048 Tests: 145 Failed: 8) > > > Failed tests: 68-69, 75-76, 86, 98, 102, 139 > > > Non-zero exit status: 8 > > > t/imapd.t (Wstat: 256 Tests: 186 Failed: 1) > > > Failed test: 183 > > > Non-zero exit status: 1 > > > t/nntpd.t (Wstat: 256 Tests: 110 Failed: 1) > > > Failed test: 104 > > > Non-zero exit status: 1 (I'm that one) So instead of giving the output of prove -bvw t/nntpd.t (I couldn't reproduce manually..), I'll go straight to the solution: it looks like there's a problem with the chattr fallback when btrfs is present on the machine (to disable cow, lib/PublicInbox/NDC_PP.pm) I had initial messages about chattr not being found; adding chattr to the build inputs gave another error message that chattr +C doesn't work on tmpfs. The problem here seems to be that the fallback here just checks /proc/self/mounts and calls chattr "just in case", saying it's ok if it fails, but some tests probably rely on stderr output being empty? If so it looks more like a test problem than anything else to me, but perhaps the fallback path could be a bit more prudent in its calling of chattr (or simply try silencing its output?) For testing, being a bit forceful and removing that chattr call made all the tests pass for me. Now the $100 question is I don't know why the Inline::C version wasn't used; this one properly calls statfs() and does the ioctl directly only if required so would have worked. As far as I can see we're installing Inline::C in the build chroot, I see it in the PERL5LIB paths; and make/gcc are also available so it probably should work. Is there a way to check that? > Sorry for the problems.... > > I wonder if it's a missing dependency that the tests forget to > account for... Can they run the tests individually using "prove" > and show more output? > > e.g.: make && prove -bvw t/nntpd.t > > Also, "./lei.sh sucks" will dump the relevant deps+versions > (either Inline::C or Socket::MsgHdr is required for lei to work) Here's what this gives me: --- lei 1.7.0 perl 5.34.0 / Linux 5.10.81 / x86_64 ptrsize=8 nproc=4 git 2.33.1 / JSON::PP 4.06 SQLite 3.32.3, DBI 1.643, DBD::SQLite 1.66 Xapian 1.4.18, bindings: Search::Xapian 1.2.25.4 public-inbox blob OIDs of loaded features: 2c9c4395c02edbb86683573612f5b454115ce719 PublicInbox/Address.pm 0f002e5e62c260fbb5078bbbb25c67b1468fbbec PublicInbox/Config.pm bacc9cdda12498abbb0ada5d2a2e2faec10190f2 PublicInbox/ContentHash.pm bf8c4466218a37a43688aaaca2b063c65e98c86e PublicInbox/DS.pm 9206da9cb710186462ffefd10cb4af0764f7e615 PublicInbox/DirIdle.pm 485f637a3e7b41a9117cb37a9acfd8e7649dadfa PublicInbox/Eml.pm 80fc7364fc9a1943759585c30487ab0c5d36ca0c PublicInbox/EmlContentFoo.pm 309f80dbfcf001e51f130cddee1a14ee38303dd3 PublicInbox/Git.pm 3e299448b334629fb6427e7df02976381ec19242 PublicInbox/IPC.pm 60ce7b66419b3ee0e9ad2dca4b93644497539378 PublicInbox/Import.pm ffe26a44ab3d9e38d46447fd69543e0a9d4e5c8d PublicInbox/In2Tie.pm 1579d500136a8f814f532275f486bc117c3e0441 PublicInbox/Inbox.pm 887025de5dedb608576b2263b4f20c0596fce8e5 PublicInbox/LEI.pm 30bb1a4579c74d8245d1eec7b66b9687bd526e71 PublicInbox/LeiExternal.pm fa0e78667b3b051cdf3dda96d0b8fba24e1f19f2 PublicInbox/LeiHelp.pm 352ee60131aaf6bc9baabfbd4ae735c4a0f06568 PublicInbox/LeiQuery.pm 8e866fc96655a9dd5a6342de43217da0a57e14f5 PublicInbox/LeiSucks.pm 7cedc3493f872df866755e1c2c9d5d545df7c319 PublicInbox/Listener.pm 0ee2a8bd60bd964a13c0c24f958f43e0ca9a6b5b PublicInbox/Lock.pm f82194a347b867aa22d416b75576e3ac624527ee PublicInbox/MDA.pm 35b517e09f5d12ff264bcd0bd8afe77f1b96e69d PublicInbox/MID.pm dd28417b70c043b092f231147d82580ba94e31cf PublicInbox/MsgIter.pm 5ee087fd50fea18612798d094046b4ada12d16e1 PublicInbox/MsgTime.pm 615bc450998beea1e82606db35c0d456450d966b PublicInbox/OnDestroy.pm 30ad949dd027153f30835f928c962d00e5d9f82a PublicInbox/Over.pm 4c434566d31f90104d921b76adf1844a9d6aefde PublicInbox/PktOp.pm 97e9c268f8d1e91aa0f8c1225112dc42cd623768 PublicInbox/ProcessPipe.pm 523003b3c269ae42e9ee210e91415b2846735daa PublicInbox/Search.pm 81e5a1b1dd88e99d25bf37aadbb1f520e8a6503f PublicInbox/Sigfd.pm 260ce6bb065e930adfd9030d50f5a06359e809ce PublicInbox/Smsg.pm 6ca1ca2a0be3538b3cb9dc663422698b6b94952b PublicInbox/Spawn.pm c00385b94db84b63facf7a8d57296ac76b3b1421 PublicInbox/Syscall.pm 3040dd77d457d02f4517088edcad4f4cf858228f PublicInbox/Tmpfile.pm 950bd17052a569b7e6792f875791d801525d821d PublicInbox/WQWorker.pm Let us know how it sucks! Please include the above and any other relevant information when sending plain-text mail to us at: meta@public-inbox.org -- archives: https://public-inbox.org/meta/ --- > > Nix being Nix we can assume the exact same code is used for all, > > but our systems are different (eg. CPU(s), filesystem(s), etc.) > > Perhaps you guys can compare installed package lists easily and > help narrow it down? There's a lot of optional stuff in > public-inbox since we try to support some old systems and users > who don't want extra dependencies; but yes, it gets difficult to > support so many possible combinations. That's the thing with nixos, in theory the package list really should be identical, so we're down to environmental differences (like me and btrfs present in /proc/self/mounts). The build happens in a chroot with just the "input" packages present (which also incidentally makes it somewhat harder to debug, I couldn't reproduce my error manually...). > > One reviewer and a bot fail with: > > > t/lei_to_mail.t .............. 1/? Use of uninitialized value in open at t/lei_to_mail.t line 263. > > > Bailout called. Further testing stopped: No such file or directory > > > FAILED--Further testing stopped: No such file or directory (waiting to see if the ones who produce this can give more details about their environment) Thanks, -- Dominique Martinet | Asmadeus