unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#25988: python2-urwid tests fail
@ 2017-03-05 23:08 Maxim Cournoyer
  2017-03-06  0:51 ` Leo Famulari
  2017-04-19 15:20 ` Maxim Cournoyer
  0 siblings, 2 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2017-03-05 23:08 UTC (permalink / raw)
  To: 25988


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

Hi,

I'm currently facing test failures from the python2-urwid package
required to build wicd. The errors are:


[-- Attachment #1.2: Type: text/plain, Size: 2995 bytes --]

test_run (urwid.tests.test_event_loops.SelectEventLoopTest) ... ok

======================================================================
FAIL: test_horizontal_resize (urwid.tests.test_vterm.TermTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 169, in test_horizontal_resize
    + '3-' + ' ' * 76 + '-4')
  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 120, in expect
    self.assertEqual(got, what, desc)
AssertionError: Expected:
'1-                                                                            -2\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4'
Got:
'Traceback (most recent call last):\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/vterm.py", l\nine 1389, in spawn\n    self.command()\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_v\nterm.py", line 44, in __call__\n    data = os.read(self.reader, 1024)\nOSError: [Errno 4] Interrupted system call'

======================================================================
FAIL: test_vertical_resize (urwid.tests.test_vterm.TermTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 186, in test_vertical_resize
    self.expect('\n' * (y - 2) + '3-' + ' ' * 76 + '-4', desc)
  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 120, in expect
    self.assertEqual(got, what, desc)
AssertionError: try to rescale to 80x22.
Expected:
'\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4'
Got:
'\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4\n\nTraceback (most recent call last):\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/vterm.py", l\nine 1389, in spawn\n    self.command()\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_v\nterm.py", line 44, in __call__\n    data = os.read(self.reader, 1024)\nOSError: [Errno 4] Interrupted system call'

----------------------------------------------------------------------
Ran 278 tests in 1.073s

FAILED (failures=2)
phase `check' failed after 5.2 seconds
builder for `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed with exit code 1
@ build-failed /gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv - 1 builder for `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed with exit code 1
guix build: error: build failed: build of `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed

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


I'm using GuixSD with ext4 partition and kernel Linux 4.9.5. The
package definitions are from git master. The system itself is using git
master from about a month ago.

lfam or #guix said he could reproduce the issue with Linux 4.9.13 on a
ext4 fileystem but not on a btrfs one.

I could not reproduce the issue myself on an Ubuntu 14.04 running Linux
4.2.0 with a ext4 partition .

Maxim

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#25988: python2-urwid tests fail
  2017-03-05 23:08 bug#25988: python2-urwid tests fail Maxim Cournoyer
@ 2017-03-06  0:51 ` Leo Famulari
  2017-04-19 15:20 ` Maxim Cournoyer
  1 sibling, 0 replies; 6+ messages in thread
From: Leo Famulari @ 2017-03-06  0:51 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 25988

[-- Attachment #1: Type: text/plain, Size: 624 bytes --]

On Sun, Mar 05, 2017 at 03:08:53PM -0800, Maxim Cournoyer wrote:
> I'm using GuixSD with ext4 partition and kernel Linux 4.9.5. The
> package definitions are from git master. The system itself is using git
> master from about a month ago.
> 
> lfam or #guix said he could reproduce the issue with Linux 4.9.13 on a
> ext4 fileystem but not on a btrfs one.

It's the other way around: I can build it on btrfs but not on ext4.

Also, I noticed yesterday that some of the urwid builds failed on Hydra.
I restarted them, and now Hydra has built both python-urwid and
python2-urwid for all 4 architectures on Linux.

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#25988: python2-urwid tests fail
  2017-03-05 23:08 bug#25988: python2-urwid tests fail Maxim Cournoyer
  2017-03-06  0:51 ` Leo Famulari
@ 2017-04-19 15:20 ` Maxim Cournoyer
  2017-04-19 21:22   ` Ludovic Courtès
  1 sibling, 1 reply; 6+ messages in thread
From: Maxim Cournoyer @ 2017-04-19 15:20 UTC (permalink / raw)
  To: 25988

[-- Attachment #1: Type: text/plain, Size: 3696 bytes --]

Hi,

Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:

> Hi,
>
> I'm currently facing test failures from the python2-urwid package
> required to build wicd. The errors are:
>
> test_run (urwid.tests.test_event_loops.SelectEventLoopTest) ... ok
>
> ======================================================================
> FAIL: test_horizontal_resize (urwid.tests.test_vterm.TermTest)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 169, in test_horizontal_resize
>     + '3-' + ' ' * 76 + '-4')
>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 120, in expect
>     self.assertEqual(got, what, desc)
> AssertionError: Expected:
> '1-                                                                            -2\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4'
> Got:
> 'Traceback (most recent call last):\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/vterm.py", l\nine 1389, in spawn\n    self.command()\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_v\nterm.py", line 44, in __call__\n    data = os.read(self.reader, 1024)\nOSError: [Errno 4] Interrupted system call'
>
> ======================================================================
> FAIL: test_vertical_resize (urwid.tests.test_vterm.TermTest)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 186, in test_vertical_resize
>     self.expect('\n' * (y - 2) + '3-' + ' ' * 76 + '-4', desc)
>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 120, in expect
>     self.assertEqual(got, what, desc)
> AssertionError: try to rescale to 80x22.
> Expected:
> '\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4'
> Got:
> '\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4\n\nTraceback (most recent call last):\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/vterm.py", l\nine 1389, in spawn\n    self.command()\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_v\nterm.py", line 44, in __call__\n    data = os.read(self.reader, 1024)\nOSError: [Errno 4] Interrupted system call'
>
> ----------------------------------------------------------------------
> Ran 278 tests in 1.073s
>
> FAILED (failures=2)
> phase `check' failed after 5.2 seconds
> builder for `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed with exit code 1
> @ build-failed /gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv - 1 builder for `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed with exit code 1
> guix build: error: build failed: build of `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed


The same problem occurred again today while attempting to reconfigure my
system (same errors, while attempting to build the same python2-urwid
dependency of wicd).

I wonder why there are so such failures on hydra
(https://hydra.gnu.org/job/gnu/master/python2-urwid-1.3.1.x86_64-linux). I
also wonder why my system is trying to build this package if it already
exists on Hydra. I have both the bayfront & gnu mirror substitute
servers enabled.

Maxim

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#25988: python2-urwid tests fail
  2017-04-19 15:20 ` Maxim Cournoyer
@ 2017-04-19 21:22   ` Ludovic Courtès
  2017-04-30  6:44     ` bug#25988: [PATCH] " Maxim Cournoyer
  0 siblings, 1 reply; 6+ messages in thread
From: Ludovic Courtès @ 2017-04-19 21:22 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 25988

Hi Maxim,

Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:

> Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
>
>> Hi,
>>
>> I'm currently facing test failures from the python2-urwid package
>> required to build wicd. The errors are:
>>
>> test_run (urwid.tests.test_event_loops.SelectEventLoopTest) ... ok
>>
>> ======================================================================
>> FAIL: test_horizontal_resize (urwid.tests.test_vterm.TermTest)
>> ----------------------------------------------------------------------
>> Traceback (most recent call last):
>>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 169, in test_horizontal_resize
>>     + '3-' + ' ' * 76 + '-4')
>>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 120, in expect
>>     self.assertEqual(got, what, desc)
>> AssertionError: Expected:
>> '1-                                                                            -2\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4'
>> Got:
>> 'Traceback (most recent call last):\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/vterm.py", l\nine 1389, in spawn\n    self.command()\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_v\nterm.py", line 44, in __call__\n    data = os.read(self.reader, 1024)\nOSError: [Errno 4] Interrupted system call'
>>
>> ======================================================================
>> FAIL: test_vertical_resize (urwid.tests.test_vterm.TermTest)
>> ----------------------------------------------------------------------
>> Traceback (most recent call last):
>>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 186, in test_vertical_resize
>>     self.expect('\n' * (y - 2) + '3-' + ' ' * 76 + '-4', desc)
>>   File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_vterm.py", line 120, in expect
>>     self.assertEqual(got, what, desc)
>> AssertionError: try to rescale to 80x22.
>> Expected:
>> '\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4'
>> Got:
>> '\n\n\n\n\n\n\n\n\n\n\n3-                                                                            -4\n\nTraceback (most recent call last):\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/vterm.py", l\nine 1389, in spawn\n    self.command()\n  File "/tmp/guix-build-python2-urwid-1.3.1.drv-0/urwid-1.3.1/urwid/tests/test_v\nterm.py", line 44, in __call__\n    data = os.read(self.reader, 1024)\nOSError: [Errno 4] Interrupted system call'
>>
>> ----------------------------------------------------------------------
>> Ran 278 tests in 1.073s
>>
>> FAILED (failures=2)
>> phase `check' failed after 5.2 seconds
>> builder for `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed with exit code 1
>> @ build-failed /gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv - 1 builder for `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed with exit code 1
>> guix build: error: build failed: build of `/gnu/store/d3nn0jqa41d7i28qvs78np7mcii2ljn2-python2-urwid-1.3.1.drv' failed
>
>
> The same problem occurred again today while attempting to reconfigure my
> system (same errors, while attempting to build the same python2-urwid
> dependency of wicd).
>
> I wonder why there are so such failures on hydra
> (https://hydra.gnu.org/job/gnu/master/python2-urwid-1.3.1.x86_64-linux). I
> also wonder why my system is trying to build this package if it already
> exists on Hydra. I have both the bayfront & gnu mirror substitute
> servers enabled.

I’ve seen the failure once and it succeeded on a subsequent rebuild, so
I came to the conclusion that this is a non-deterministic failure.  That
would explain why the problem doesn’t show up on Hydra.

Does it fail systematically for you?

Thanks,
Ludo’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#25988: [PATCH] bug#25988: python2-urwid tests fail
  2017-04-19 21:22   ` Ludovic Courtès
@ 2017-04-30  6:44     ` Maxim Cournoyer
  2017-05-01 14:47       ` Ludovic Courtès
  0 siblings, 1 reply; 6+ messages in thread
From: Maxim Cournoyer @ 2017-04-30  6:44 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 25988


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

Hi Ludovic,

ludo@gnu.org (Ludovic Courtès) writes:

[...]

>
> I’ve seen the failure once and it succeeded on a subsequent rebuild, so
> I came to the conclusion that this is a non-deterministic failure.  That
> would explain why the problem doesn’t show up on Hydra.
>
> Does it fail systematically for you?
>

Yes, it does! I can re-run the `guix system reconfigure ...' as
many times as I want and always get the same two failures. However, if I
keep the failed derivation and run the tests manually, I've found that
using 'strace' will cause more failures/errors; so it seems
non-deterministic; maybe a racing issue?

I've opened an issue on their GitHub tracker
(https://github.com/urwid/urwid/issues/230). Interestingly the same
tests run fine with Python 3, with or without using strace.

I've attached a patch that disables the problematic vterm tests, only
for the python2-urwid package.

Maxim


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-Disable-vterm-tests-for-python2-urwid.patch --]
[-- Type: text/x-patch, Size: 1379 bytes --]

From 56e769bee4a47136e0eee251c87a58b8359a8611 Mon Sep 17 00:00:00 2001
From: Maxim Cournoyer <maxim.cournoyer@gmail.com>
Date: Sun, 30 Apr 2017 14:55:47 +0900
Subject: [PATCH] gnu: Disable vterm tests for python2-urwid

* gnu/packages/python.scm (python2-urwid)['delete-test_vterm]: Add phase to
delete problematic test modules.
---
 gnu/packages/python.scm | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 22fadea275..c6c0978038 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -5509,7 +5509,18 @@ features useful for text console applications.")
     (license license:lgpl2.1+)))
 
 (define-public python2-urwid
-  (package-with-python2 python-urwid))
+  (let ((python2-urwid (package-with-python2 python-urwid)))
+    (package
+      (inherit python2-urwid)
+      (arguments
+       (append
+        '(#:phases
+          (modify-phases %standard-phases
+            ;; Disable the vterm tests because of non-deterministic failures
+            ;; with Python 2. See https://github.com/urwid/urwid/issues/230.
+            (add-after 'unpack 'delete-test_vterm.py
+              (delete-file "urwid/tests/test_vterm.py"))))
+        (package-arguments python-urwid))))))
 
 (define-public python-openid
   (package
-- 
2.12.0


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

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* bug#25988: [PATCH] bug#25988: python2-urwid tests fail
  2017-04-30  6:44     ` bug#25988: [PATCH] " Maxim Cournoyer
@ 2017-05-01 14:47       ` Ludovic Courtès
  0 siblings, 0 replies; 6+ messages in thread
From: Ludovic Courtès @ 2017-05-01 14:47 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 25988-done

Hi Maxim,

Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:

> ludo@gnu.org (Ludovic Courtès) writes:
>
> [...]
>
>>
>> I’ve seen the failure once and it succeeded on a subsequent rebuild, so
>> I came to the conclusion that this is a non-deterministic failure.  That
>> would explain why the problem doesn’t show up on Hydra.
>>
>> Does it fail systematically for you?
>>
>
> Yes, it does! I can re-run the `guix system reconfigure ...' as
> many times as I want and always get the same two failures. However, if I
> keep the failed derivation and run the tests manually, I've found that
> using 'strace' will cause more failures/errors; so it seems
> non-deterministic; maybe a racing issue?
>
> I've opened an issue on their GitHub tracker
> (https://github.com/urwid/urwid/issues/230). Interestingly the same
> tests run fine with Python 3, with or without using strace.
>
> I've attached a patch that disables the problematic vterm tests, only
> for the python2-urwid package.

Sounds reasonable to me.

> Maxim
>
> From 56e769bee4a47136e0eee251c87a58b8359a8611 Mon Sep 17 00:00:00 2001
> From: Maxim Cournoyer <maxim.cournoyer@gmail.com>
> Date: Sun, 30 Apr 2017 14:55:47 +0900
> Subject: [PATCH] gnu: Disable vterm tests for python2-urwid
>
> * gnu/packages/python.scm (python2-urwid)['delete-test_vterm]: Add phase to
> delete problematic test modules.

Applied, thanks!

Ludo’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-05-01 14:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-05 23:08 bug#25988: python2-urwid tests fail Maxim Cournoyer
2017-03-06  0:51 ` Leo Famulari
2017-04-19 15:20 ` Maxim Cournoyer
2017-04-19 21:22   ` Ludovic Courtès
2017-04-30  6:44     ` bug#25988: [PATCH] " Maxim Cournoyer
2017-05-01 14:47       ` Ludovic Courtès

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).