search

uwiger / gproc

Extended process registry for Erlang
Apache License 2.0
1.07k stars 232 forks source link

gproc_dist_tests: dist_test_...*timed out* #51

Closed matwey closed 3 years ago

matwey commented 10 years ago

Hi, I am suffering an issue similar to #44 on the latest master. See the eunit log: http://susepaste.org/27740609

The difference is that failure is reproduced only approx 3 times per 50 runs (on my PC). I've just been running rebar eunit in a loop. Increasing timeout up to 240 seems to make failure probability as low as no failures per 50 runs.

Not sure if that test is deterministic...

uwiger commented 10 years ago

Sorry about the delay. I have made some changes - most importantly, changed the 'inparallel' macro to 'inorder'. I think the nondeterminism hides somewhere in the test code, but haven't been able to pin it down. After making minor changes here and there, I could no longer reproduce the error, but the parallel runs also make it harder to debug failing tests.

matwey commented 10 years ago

Hi,

Sorry to say, but I've found yet another(?) stochastic issue after your fix: Could you check whether It is the same? http://susepaste.org/51044195

uwiger commented 10 years ago

Yes, there seems to be some timing issue with gproc_monitor.

Hopefully, as the uw-standby-monitors branch is starting to look stable (issue #53), I will be able to deprecate gproc_monitor in favor of gproc:monitor(Key, follow).

uwiger commented 10 years ago

I did tweak gproc_monitor a bit. At least on my laptop, this seems to have made it harder to reproduce the problem. Pushed to master.

uwiger commented 9 years ago

Have you seen any other intermittent failures?

matwey commented 9 years ago

With the latest master I still sometimes see: http://susepaste.org/25878998

uwiger commented 9 years ago

Ok, I'll keep the issue open then.

uwiger commented 9 years ago

I've just merged some changes that should at least take care of the timeout in t_follow_monitor/1.

manuel-rubio commented 3 years ago

if @matwey isn't complaining it's meaning it's fixed. @uwiger I think you can close the ticket.

matwey commented 3 years ago

Lets close this. I'll reopen if required