guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests: Don't check file-systems in container tests.


From: Ludovic Courtès
Subject: Re: [PATCH] tests: Don't check file-systems in container tests.
Date: Mon, 14 Nov 2016 15:42:47 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Marius Bakke <address@hidden> skribis:

> Ludovic Courtès <address@hidden> writes:
>
>> Hi,
>>
>> Andy Patterson <address@hidden> skribis:
>>
>>> The containers test was hanging for me, and this patch fixed the
>>> problem.
>>
>> [...]
>>
>>> From 945ad44acf489b7f3a398d4ab739ec2b48477502 Mon Sep 17 00:00:00 2001
>>> From: Andy Patterson <address@hidden>
>>> Date: Sat, 12 Nov 2016 22:10:01 -0500
>>> Subject: [PATCH] tests: Don't check file-systems in container tests.
>>>
>>> * tests/containers.scm ("call-with-container, mnt namespace"): Don't
>>> check file-system in 'call-with-container' call.
>>> * tests/containers.scm
>>> ("call-with-container, mnt namespace, wrong bindmount"): Likewise.
>>> ---
>>>  tests/containers.scm | 6 ++++--
>>>  1 file changed, 4 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/tests/containers.scm b/tests/containers.scm
>>> index ccd122a..745b56b 100644
>>> --- a/tests/containers.scm
>>> +++ b/tests/containers.scm
>>> @@ -84,7 +84,8 @@
>>>     (call-with-container (list (file-system
>>>                                  (device "none")
>>>                                  (mount-point "/testing")
>>> -                                (type "tmpfs")))
>>> +                                (type "tmpfs")
>>> +                                (check? #f)))
>>
>> Do you know exactly how/why it was hanging?  I imagine
>> ‘mount-file-system’ would try to invoke fsck.tmpfs, which doesn’t exist,
>> thus we get a REPL, which hangs forever.
>>
>> I guess the real question is why I didn’t experience it, hmm…
>
> This is failing for me on master, I think the guix development snapshot
> needs to be updated to include this fix.

I believe commit 198eac2bca075d0e71e504f1e8c46fddc62171bb solves that.

I don’t think the regression has anything to do with it the recent
changes though.  It’s just that, contrary to what was intended in
package-management.scm, this container test wasn’t skipped, and it
happens to fail surprisingly in the build environment (for reasons that
one of us will undoubtedly elucidate so we don’t remain frustrated!).

Thanks,
Ludo’.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]