neurobagel / documentation

The neurobagel documentation
http://neurobagel.org/
MIT License
10 stars 6 forks source link

Required version of Docker Compose is too old for secrets #215

Closed alyssadai closed 3 months ago

alyssadai commented 3 months ago

Description

We currently specify a minimum Docker Compose version of 2.0.0, but this version actually (apparently) doesn't support Docker secrets set by environment variables.

Steps to reproduce

If you try to spin up a local node using Docker Compose following our docs with a version around 2.0.0 (e.g., 2.2.3), you will get an error on launch:

secrets.db_admin_password Additional property environment is not allowed

Steps to fix

Based on https://docs.docker.com/compose/releases/release-notes/#270, it looks like support for environment secrets, including during the build step, was introduced in v2.7.0. We need to update the minimum Docker Compose version to match.

Relevant documentation pages

http://127.0.0.1:8000/getting_started/#docker-and-docker-compose

Additional details

No response