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

A strange behavior when running anycable-go (systemd restarts) #10

Closed
GabrielMalakias opened this issue Sep 13, 2017 · 4 comments
Closed
Assignees
Labels

Comments

@GabrielMalakias
Copy link

GabrielMalakias commented Sep 13, 2017

I've just published my system but sometimes systemd restarts this server throwing this error:

Sep 13 03:08:29 debian-yggdrasil bash[5523]: Broadcast to stream sensor_1: {"user":"external","message":"22.97","sid":98016}
Sep 13 03:08:29 debian-yggdrasil bash[5523]: panic: send on closed channel
Sep 13 03:08:29 debian-yggdrasil bash[5523]: goroutine 8 [running]:
Sep 13 03:08:29 debian-yggdrasil bash[5523]: main.(*Hub).run(0xa99d40)
Sep 13 03:08:29 debian-yggdrasil bash[5523]: #011/Users/palkan/lib/go/src/github.com/anycable/anycable-go/hub.go:107 +0x9f8
Sep 13 03:08:29 debian-yggdrasil bash[5523]: created by main.main
Sep 13 03:08:29 debian-yggdrasil bash[5523]: #011/Users/palkan/lib/go/src/github.com/anycable/anycable-go/server.go:176 +0x281
Sep 13 03:08:29 debian-yggdrasil systemd[1]: anycable-go.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Sep 13 03:08:29 debian-yggdrasil systemd[1]: anycable-go.service: Unit entered failed state.
Sep 13 03:08:29 debian-yggdrasil systemd[1]: anycable-go.service: Failed with result 'exit-code'.
Sep 13 03:08:39 debian-yggdrasil systemd[1]: anycable-go.service: Service hold-off time over, scheduling restart.
@GabrielMalakias GabrielMalakias changed the title An strange behavior when running anycable-go (systemd restarts) A strange behavior when running anycable-go (systemd restarts) Sep 13, 2017
@palkan palkan added the bug label Sep 19, 2017
@palkan palkan self-assigned this Sep 19, 2017
@palkan palkan closed this as completed in c6173dd Sep 27, 2017
@GabrielMalakias
Copy link
Author

Thank you!

@palkan
Copy link
Member

palkan commented Sep 27, 2017

Note: I haven't tried it in production yet; but I've added a test for this small change, everything looks good)

@palkan
Copy link
Member

palkan commented Sep 27, 2017

Binaries uploaded, see Downloads page

@GabrielMalakias
Copy link
Author

Ok.. That's fine, I'm testing it in a personal project, so the whole project is in beta. ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants