Skip to content
This repository has been archived by the owner on Jul 9, 2023. It is now read-only.

proposal: include gettext #18

Closed
iAnanich opened this issue Sep 1, 2020 · 3 comments
Closed

proposal: include gettext #18

iAnanich opened this issue Sep 1, 2020 · 3 comments
Labels

Comments

@iAnanich
Copy link

iAnanich commented Sep 1, 2020

gettext allows me to use envsubst which is nice for configuring rendering docker-compose.yml with external secrets, networks etc as their names better to be stored as CI variables rather as hardcoded value in docker-compose.yml.

@tiangolo
Copy link
Owner

You can use environment variables in docker-compose files: https://docs.docker.com/compose/compose-file/#service-configuration-reference

In any case, there are many different tools that would work for each individual use case, so I would prefer to keep this as simple as possible. If you find yourself needing some other tools frequently, I think it would be better to create a new re-usable image that works best for you.

@iAnanich
Copy link
Author

Fair enough )
Actually, I'm using env vars lately. The reason behind gettext was to make some attributes changeable (like network name, volume name, secret name). But further digging into Docker Compose documentation and newer versions revealed that all of them can have external name specified with env var while keeping internal (in context of single docker compose) name constant.

@github-actions
Copy link

Assuming the original issue was solved, it will be automatically closed now. But feel free to add more comments or create new issues.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants