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

Couldn't cleanly terminate all actors in 10 seconds! #79

Closed
dylangrafmyre opened this issue Jan 20, 2017 · 5 comments
Closed

Couldn't cleanly terminate all actors in 10 seconds! #79

dylangrafmyre opened this issue Jan 20, 2017 · 5 comments

Comments

@dylangrafmyre
Copy link

dylangrafmyre commented Jan 20, 2017

This gem was added to our Gemfile ldclient-rb 2.0.3. Gemspec dependency ld-celluloid-eventsource adds celluliod-io 0.17.3 which depends on celluloid 0.17.3.

Running rails, rake, rspec commands cause slowness with termination of the task.

rake db:migrate
Running via Spring preloader in process 44805
== 20170120043022 EnsureGuestListingsExistForAllInvitations: migrating ========
== 20170120043022 EnsureGuestListingsExistForAllInvitations: migrated (3.1553s)

Annotated (1): app/models/listing_geo_data.rb
E, [2017-01-20T00:32:46.013847 #44805] ERROR -- : Couldn't cleanly terminate all actors in 10 seconds!

Please see: celluloid/celluloid#696
This was a regression in celluloid and there should be a fix in 0.17.4 and 0.18.0.pre.

Thanks for looking into possibly updating the ld-celluloid-eventsource dependency on celluloid.

@jkodumal
Copy link
Contributor

Thanks @dylangrafmyre -- we'd been waiting for a final 0.18.0 celluloid release, but we'll give 0.18.0.pre a try.

@colmdoyle
Copy link
Contributor

@jkodumal
Copy link
Contributor

jkodumal commented Mar 6, 2017

@colmdoyle We're testing this out now-- we need to run it through our integration suite and ensure it passes before we release. We'll keep you posted.

@jkodumal
Copy link
Contributor

jkodumal commented Mar 7, 2017

@colmdoyle Things are looking good, and we anticipate that we'll be releasing an updated version tomorrow (Wednesday).

@jkodumal
Copy link
Contributor

jkodumal commented Mar 8, 2017

This should be addressed in v 2.1.0

@jkodumal jkodumal closed this as completed Mar 8, 2017
eli-darkly added a commit that referenced this issue Aug 30, 2018
fix event generation for a prerequisite that is off
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

3 participants