-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Coroutine raised StopIteration ci-runner error #62
Comments
Saw the exact failure again: https://github.com/valefar-on-discord/ethstaker-deposit-cli/actions/runs/9438041063/job/25994595794 |
I believe I just reproduced this issue on https://github.com/remyroy/ethstaker-deposit-cli/actions/runs/9451804698/job/26033609500 |
This seems to be related to the use of pipes in asyncio.create_subprocess_shell. The only tests failing are those that uses pipes (only 2, |
">Is there a way/workaround to get this fixed in python 3.8 and python 3.9? The patch should be easy enough to back port, but that's not something we'll do because this is not a security issue. In your own code you could monkey patch multiprocessing, in particular by patching the Listener class to use a larger backlog." |
I think we can reasonably wontfix this. It doesn't impact running deposit on macOS with Python 3.9, just the test suite. If we're keen we can exclude two tests on that platform. Trying out a more granular skip in #104 |
I'm closing this as wontfix: The workaround of skipping those two tests on macOS Python 3.9 seems reasonable to me, and Python 3.9 is not getting the fix for it back-ported. |
After landed #48 I noticed the ci-runner failed with a non-assertion error but a
RuntimeError: coroutine raised StopIteration
https://github.com/valefar-on-discord/ethstaker-deposit-cli/actions/runs/9331869782/job/25687038382
I tried to reproduce this locally to no avail and I haven't seen it since from git actions but something to keep a look out for
The text was updated successfully, but these errors were encountered: