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
czmq selftests hang or die with zeromq-2 #1825
Comments
Configured the codebases about same way as travis does it :
|
Yeah I mentioned this here: #1762 (comment) |
Yes, "screenshots" above are from my local debian8 vm. So it is reproducible off-travis too. |
Oh that's great, I couldn't manage to reproduce it on my deb9 laptop. Any idea what the problem is? |
Nope, did not dig into code so far and not sure when/if I would - so feel free to overtake ;) |
Tried a while back but could not repro - what's the config on the VM? Amd64 or i386? Cpu / ram? And what's the host cpu spec? |
X86-64, I think 2 or 4 virtual cores and 2 or 4gb RAM allocated - not near it today. |
Intel i7-6820HQ @ 2.70 GHz on host; 2 vcpu and 2gb on VM. |
I'd hazard a guess that there was some change in another piece of equation, e.g. libsodium or something further along? |
…ller Solution: remove it. It is a backward-compat macro to use only with the zmq_poll API, which used to take nanoseconds until libzmq 3.x where it was changed to milliseconds. In the zbeacon test it was used with zpoller_wait, which itself used it, which means that when building with libzmq 2.x the timeout was twice multiplied by 1000 and caused the test to hang. Also remove it from the zmq_poller usage, which is new and has always taken milliseconds. Fixes zeromq#1825
Several czmq selftests just do not pass with zeromq-2, some others time out over the limit Travis CI gives to the job (10 minutes of quietness in the log, 30 minutes overall).
If the corresponding features are not supported in the old zeromq release, they should get excluded from the test (e.g. like zproc skips itself for zeromq older than v4). If they are expected to work, these tests and/or https://github.com/zeromq/zeromq2-x Git HEAD codebase should be fixed, I guess.
The text was updated successfully, but these errors were encountered: