search

Netflix / lemur-docker

Docker files for the Lemur certificate orchestration tool
170 stars 83 forks source link

fix old docker-compose #53

Closed sergerdn closed 3 years ago

sergerdn commented 3 years ago

fixes #52

jtschladen commented 3 years ago

Thanks @sergerdn. I originally added the conditions due to intermittent timing issues I was seeing in service ready order, but at the moment your proposed version isn't giving me any trouble. I'm going to go ahead and merge this for now, but I will revisit (using a v3 recommended pattern) if we see ordering problems again.

sergerdn commented 3 years ago

Thanks, @jtschladen. Have a look https://github.com/vishnubob/wait-for-it.