whisklabs / docker-it-scala

Docker integration testing kit with Scala
MIT License
431 stars 91 forks source link

Concurrency issues on logs #111

Open andreaTP opened 6 years ago

andreaTP commented 6 years ago

I do not think that it will be solved in this version but I open the ticket at least to track the issue for reference.

Given: https://github.com/whisklabs/docker-it-scala/blob/master/impl/spotify/src/main/scala/com/whisk/docker/impl/spotify/SpotifyDockerCommandExecutor.scala#L146

https://github.com/whisklabs/docker-it-scala/blob/master/impl/spotify/src/main/scala/com/whisk/docker/impl/spotify/SpotifyDockerCommandExecutor.scala#L168

The DockerReadyChecker can eventually close the Stream that is used by LogLineReceiver.

Rough workaround is to not use DockerReadyChecker.LogLinesContains along with LogLineReceiver.