-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
support --env-file in docker-compose run command #8379
Comments
@chris-crone @ndeloof Hi guys, I would like to work on this issue. I am not new to Go lang. I have got a question. If I pass |
In meantime I've created a draft PR which assumes that if you pass --env-file from CLI it overrides the env-file specified in docker-compose.yaml file. I will update PR as per your suggestion if we should append them. |
@chris-crone @ndeloof @ulyssessouza Friendly ping. |
I would think would want to append the --env-file(s) passed via the cli to any included in the service's definition. appending would override any previously defined keys. |
I'm surprised this issue hasn't more discussions. Just had this case in a project and that's a deal breaker, will have to generate a huge I think there's a misconception with the original Now the topic is stuck because it may conflict with the original What about a |
That's indeed sad this flag name was selected, but this is the legacy we have to deal with. |
@ndeloof I have updated the PR with changes suggested. |
NOTE: this is different than the
.env
file that is read by docker-compose commands and used for substitutions in the composition file... first mentioned here: #6170 (comment)We generate an env file that we would like to include via the
docker-compose run
command. e.g.docker run
supports this flag and it would be great if we sawdocker-compose run
maintain parity.Is there planned support for this feature -- or will it be implemented in composev2 / "docker compose" ?
our generated vars file is cumbersome (with multiline values) and relatively complicated to convert to various
-e FOO=bar
flags...thanks!
The text was updated successfully, but these errors were encountered: