Skip to content
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

timeout await for agent: driver-conductor. Retrying... #338

Closed
patriknw opened this issue Apr 12, 2017 · 4 comments
Closed

timeout await for agent: driver-conductor. Retrying... #338

patriknw opened this issue Apr 12, 2017 · 4 comments

Comments

@patriknw
Copy link

After updating Akka from 1.2.3 to 1.2.4 one of our tests went haywire. Endless loop of "timeout await for agent: driver-conductor. Retrying...".

It's not reproducible every time but failed several times in our nightly tests.

Here is the test and the problem is in the shutdown after the actual successful tests: https://github.com/akka/akka/blob/master/akka-remote-tests/src/multi-jvm/scala/akka/remote/artery/AeronStreamConcistencySpec.scala#L89-L91

mjpt777 added a commit that referenced this issue Apr 12, 2017
@mjpt777
Copy link
Contributor

mjpt777 commented Apr 12, 2017

I've tried to recreate this but with no luck. Seems like there might be a race condition in the start stop logic. 1.2.4 is more efficient for startup and shutdown. I'm made a change to Agrona that should avoid any potential race conditions in this area.

@patriknw
Copy link
Author

stacks.zip

Here are some thread dumps, 2 different processes, 3 dumps for each.

mjpt777 added a commit that referenced this issue Apr 12, 2017
…r conductor as this is no longer required and can cause he driver to lockup. Issue #338.
mjpt777 added a commit that referenced this issue Apr 12, 2017
…ductor into the sender and receiver on shutdown. Issue #338.
@mjpt777
Copy link
Contributor

mjpt777 commented Apr 12, 2017

This is very helpful. There is a race on shutdown which I have addressed.

@patriknw
Copy link
Author

great, thanks

@mjpt777 mjpt777 closed this as completed Apr 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants