Bug 823458 – =dev-python/ipyparallel-7.1.0 fails tests due to time-outs

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Marek Szuba 2021-11-12 22:23:45 UTC]

See the log. Observed on riscv, not tried on amd64 yet.

[Comment 1 Marek Szuba 2021-11-12 22:24:50 UTC]

Created attachment 750852 [details]
ipyparallel-7.1.0-build.log (riscv)

[Comment 2 Larry the Git Cow 2021-11-14 08:35:54 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6d3fe771b9ff45f5c3f1fef2aff83992c5326b10

commit 6d3fe771b9ff45f5c3f1fef2aff83992c5326b10
Author:     Sam James <[e-mail]>
AuthorDate: 2021-11-14 08:35:42 +0000
Commit:     Sam James <[e-mail]>
CommitDate: 2021-11-14 08:35:42 +0000

    dev-python/ipyparallel: crank up test timeouts
    
    Closes: https://bugs.gentoo.org/823458
    Signed-off-by: Sam James <[e-mail]>

 .../files/ipyparallel-7.1.0-test-timeouts.patch    | 48 ++++++++++++++++++++++
 dev-python/ipyparallel/ipyparallel-7.1.0.ebuild    |  4 ++
 2 files changed, 52 insertions(+)

[Comment 3 Marek Szuba 2021-11-16 14:37:15 UTC]

Created attachment 751718 [details]
ipyparallel-7.1.0-build.log (riscv, post-patching)

Alas, not quite there yet:
 - TestClient::test_activate, TestClient::test_lazy_all_targets and TestClient::test_wait_for_engines still time out (it seems the patch doesn't cover these);
 - AsyncResultTest::test_wait_for_send now fails because it expected a TimeoutError and hasn't received one!

[Comment 4 Sam James 2021-11-18 05:41:52 UTC]

(In reply to Marek Szuba from comment #3)
> Created attachment 751718 [details]
> ipyparallel-7.1.0-build.log (riscv, post-patching)
> 
> Alas, not quite there yet:
>  - TestClient::test_activate, TestClient::test_lazy_all_targets and
> TestClient::test_wait_for_engines still time out (it seems the patch doesn't
> cover these);
>  - AsyncResultTest::test_wait_for_send now fails because it expected a
> TimeoutError and hasn't received one!

Hmph. Okay, looking...

[Comment 5 Larry the Git Cow 2021-11-18 05:47:22 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a4681c33b0ab7eb36b5580494e132853c91033f5

commit a4681c33b0ab7eb36b5580494e132853c91033f5
Author:     Sam James <[e-mail]>
AuthorDate: 2021-11-18 05:46:36 +0000
Commit:     Sam James <[e-mail]>
CommitDate: 2021-11-18 05:46:36 +0000

    dev-python/ipyparallel: skip problematic slow tests
    
    Trying to avoid getting into an ever-growing patch extending timeouts,
    especially given these additional tests are all based around timeouts/waiting
    in some form.
    
    We still get very good coverage without these, so it's not a problem.
    
    Closes: https://bugs.gentoo.org/823458
    Signed-off-by: Sam James <[e-mail]>

 dev-python/ipyparallel/ipyparallel-7.1.0.ebuild | 9 +++++++++
 1 file changed, 9 insertions(+)