mdouchement / wctop

wctop is a simple web-ui for monitoring local running Docker containers
MIT License
2 stars 1 forks source link

async.Start() fails to restart containers collect #4

Closed mdouchement closed 6 years ago

mdouchement commented 6 years ago

https://github.com/mdouchement/wctop/blob/master/async/container.go#L35

The web-ui is totally frozen due to no ws messages. Is there a deadlock in the backend?

== Starting server on http://0.0.0.0:8080
Remote 172.17.0.1:43422 has subscribed
Starting watching container wctop
Starting watching container postgres
Starting watching container redis
Remote 172.17.0.1:43444 has subscribed
Stopping watching container wctop
Stopping watching container postgres
Remote 172.17.0.1:44752 has subscribed
Remote 172.17.0.1:44764 has subscribed
Remote 172.17.0.1:44766 has subscribed

PS: I cannot see Stopping watching container redis in logs..

mdouchement commented 6 years ago

May be resolved by https://github.com/mdouchement/wctop/commit/631a97cb3de4325f076723ddb16c442e84d8bf91