-
-
Notifications
You must be signed in to change notification settings - Fork 399
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
Fix docker-compose.yml #1912
base: main
Are you sure you want to change the base?
Fix docker-compose.yml #1912
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/athens-research/athens/HkADHNbYvarmi3MEPTwQH73V4ir4 |
Heya @siraht thanks for taking this on. I think something went wrong though, because the whole file shows up as new in the diff and it also shows up as a branch conflict in the merge widget. I'm a bit confused about the file being incorrectly formatted though... I'd expect docker compose to complain about it, but I haven't seen any error messages or weird behaviour around What version of docker and docker compose are you on? The server we run this file on is using Docker 20.10.8, and Docker Compose 1.27.4. |
I can see issues that sound like what you describe in docker/compose#4538 and docker/compose#4305. I don't get any error locally or on the server though... I wonder if it's related to the platform. I'm on mac and the server is linux. |
I also see mentions of it working again as of docker compose 1.27: https://news.ycombinator.com/item?id=25329651 I'm very confused about this tbh... Kind expected to find some authoritative information somewhere in the official docker channels. |
@filipesilva sorry about the pull request issues, this was actually my first pull request ever! (And I think I know what I did wrong :P) Anyways, with an updated Docker installation on Ubuntu (tried on multiple installations) I could not get the docker-compose working except for the configuration here. |
The docker-compose.yml file has been incorrectly formatted for the version of compose used, for months now. I've fixed it by adding dashes in front of
depends_on
services as well as removing the deprecatedcondition
fields within thedepends_on
fields.