search

zeek / zeek-testing-cluster

External testsuite for the Zeek Cluster Controller
1 stars 2 forks source link

Various fixes (was: btest: Set COMPOSE_COMPATIBILITY=true) #16

Closed awelzel closed 2 years ago

awelzel commented 2 years ago

Using a v2 docker-compose version uses - as separator between project name and container, breaking some of the assumptions made in the scripts.

Setting COMPOSECOMPATIBILITY allows to revert to using `` as a separator.

$ docker-compose --version
Docker Compose version v2.10.2

@ckreibich - is this something you could pick up or see if there's a better way to fix?

awelzel commented 2 years ago

Hmm, I pushed two more commits and now things run locally for me.

I found grabbing logs even if we don't stop containers provided improved debug experience.

The log-archival ones seem the right and required fixes, but I'm a bit confused why they would've worked previously. Might be some script-loading-order difference.