all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Marius Bakke <mbakke@fastmail.com>
To: Mark H Weaver <mhw@netris.org>
Cc: Platoxia <platoxia@protonmail.com>, 35521@debbugs.gnu.org
Subject: bug#35521: Mariadb test suite failures on x86_64-linux
Date: Fri, 12 Jul 2019 16:58:48 +0200	[thread overview]
Message-ID: <878st3mh93.fsf@devup.no> (raw)
In-Reply-To: <87ims8mds7.fsf@devup.no>


[-- Attachment #1.1: Type: text/plain, Size: 2094 bytes --]

Marius Bakke <mbakke@fastmail.com> writes:

> Mark H Weaver <mhw@netris.org> writes:
>
>> Hi,
>>
>> Marius Bakke <mbakke@fastmail.com> writes:
>>
>>> Chris Marusich <cmmarusich@gmail.com> writes:
>>>
>>>> Hi,
>>>>
>>>> I've been encountering this failure off and on for a few weeks now, and
>>>> I'd like to help fix it.  In short, it seems like non-deterministic test
>>>> failures, to me.  I think we should gather data and report the issue
>>>> upstream, and maybe disable the offending tests in the meantime.
>>>
>>> I agree.  I notice many of these failing tests are for the TokuDB
>>> backend, which I doubt anyone is using in Guix anyway.
>>>
>>> Here is a patch that disables all tests mentioned in this report.  I
>>> would like to push it to core-updates.  Are there others?
>>
>> I'm concerned by how frequently and casually we simply disable failing
>> tests.  What is the utility of running test suites at all, if this is
>> how we respond?
>
> I had no idea this issue was so widespread until I noticed Berlins
> builders hit it more often than not.  I have not been able to reproduce
> these failures on my machines.  So it was kind of a panic reaction,
> being the person responsible for running these tests and all.
>
> Looking further into the changes between 10.1.37 and 10.1.38, I notice
> the 'tokudb.*' tests were enabled:
>
> https://github.com/MariaDB/server/commit/4c490d6df63695dc97b2c808e59954e6877d3a51
>
> Watching the build on Berlin in real time, I also see that the test
> output grind nearly to a halt while running those.
> 'tokudb.hotindex-insert-2' took 2700439 milliseconds, or 45 minutes, if
> I'm reading the test output correctly.
>
> The default test case timeout is 40 minutes (as specified in the Guix
> package), but I'm using 80 for this build (60 was insufficient).
>
> I suspect the problem is that the 'tokudb.*' tests put a lot of strain
> on the file system, which causes these other tests to fail.  It's
> interesting that disabling parallel build was insufficient though.

Update: Berlin built mariadb twice on core-updates with this patch:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: db.diff --]
[-- Type: text/x-patch, Size: 645 bytes --]

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index 6bfeaad9a2..64bc0938b6 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -753,7 +753,7 @@ Language.")
                  (with-directory-excursion "mysql-test"
                    (invoke "./mtr" "--verbose"
                            "--retry=3"
-                           "--testcase-timeout=40"
+                           "--testcase-timeout=80"
                            "--suite-timeout=600"
                            "--parallel" (number->string (parallel-job-count))
                            "--skip-test-list=unstable-tests"))

[-- Attachment #1.3: Type: text/plain, Size: 88 bytes --]


Mark, Chris: Can you try this change with MariaDB 10.1.40 and see if it
works for you?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

  parent reply	other threads:[~2019-07-12 14:59 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-01  9:18 bug#35521: Mariadb test suite failures on x86_64-linux Mark H Weaver
2019-05-01  9:47 ` Mark H Weaver
2019-07-10  6:18   ` Chris Marusich
2019-07-10 17:30     ` Marius Bakke
2019-07-10 21:32       ` Mark H Weaver
2019-07-11 20:18         ` Ludovic Courtès
2019-07-12  8:02           ` Giovanni Biscuolo
2019-07-11 22:01         ` Marius Bakke
2019-07-12  8:24           ` Giovanni Biscuolo
2019-07-12 14:58           ` Marius Bakke [this message]
2019-07-13 17:29             ` Mark H Weaver
2019-07-13 22:42               ` Marius Bakke
2019-07-14  2:35                 ` Mark H Weaver
2019-07-14 11:10                   ` Arne Babenhauserheide
2019-07-14 15:58                     ` Gábor Boskovits
2019-07-14  4:42                 ` Mark H Weaver
2019-07-14 17:17                   ` Marius Bakke
2019-07-14 18:34                     ` Mark H Weaver
2019-07-14 21:15                       ` Marius Bakke
2019-07-15 16:49                 ` Testing changes on Berlin before pushing changes? (Was: Re: bug#35521: Mariadb test suite failures on x86_64-linux) Chris Marusich
2019-07-15 18:59                   ` Marius Bakke
2019-07-16  7:19                     ` Testing changes on Berlin before pushing changes? Chris Marusich
2019-07-13 18:38             ` bug#35521: Mariadb test suite failures on x86_64-linux Mark H Weaver
2019-05-10  1:33 ` bug#35521: /gnu/store/c46sn2yfllcfi86p8227wvvr1bxssgxj-mariadb-10.1.38.drv - Failing test(s): tokudb_alter_table.hcad_all_add Platoxia

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=878st3mh93.fsf@devup.no \
    --to=mbakke@fastmail.com \
    --cc=35521@debbugs.gnu.org \
    --cc=mhw@netris.org \
    --cc=platoxia@protonmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.