-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Latest aleth fails with Solidity tests #3825
Comments
For issue
Running it twice stops |
this are 3 PRs that changed the client recently |
This seems to be specific to macOS and is probably related to timing of unix sockets. |
@axic is this fixed? |
No, still happens. |
Run this again with the official aleth 1.4.0 build and it is still happening.
|
This is on mac 10.11.6, will need to check on a newer system. |
could you print the actuall RPC response? |
This issue is older. |
Should be fixed now. |
Running aleth 1.5.2 and it still happens:
|
Now after 4-6 connections attempts aleth just quits. This leads me to believe it closes because the unix socket has shut down abruptly:
|
The actual underlying problem is that The problem is that for some reason on mac UNIX socket transmission is buffered, seemingly with 8K. This means we stop reading the response after 8K, which results in a short input in some test cases (especially where contract bytecode is returned). |
Could this also be the cause why our CI sometimes randomly fails in the RPCSession file? |
It could be. |
Is this still valid? |
I haven't had any problems recently with aleth 1.6.0 and I'm not aware of anybody else having problems - should we close this? |
I think this is fixed and obsolete and I'm closing it. Please re-open, if there's still any issues. |
Unless the RPC handling has been changed to properly handle different unix socket buffer sizes, than this problem persists. |
Ok, I'm reopening until we have confirmation that it's fixed. |
I am new to contributing here and seem to be facing this same issue. I am running Mac 10.14.5. Has anyone else had any recent issues with this? |
The tests are known to have issues on MacOS. The hope is that aleth won't be necessary at all in a few days. This will remove the need for aleth: #7010 Still, that PR has not yet been tested on MacOS... |
Can this issue be closed once #7124 is merged? |
It is not relevant anymore. |
Was fixed by #7010. |
The text was updated successfully, but these errors were encountered: