-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
docker-compose build
shouldn't complain about missing env file
#3346
Labels
Comments
Duplicate of #1973 |
Seems to me that this is not a duplicate. Separating the parse in 2 steps will not allow some env_file to be non-existent. On the phase that it is read, non-existing files will raise an error. The asked here is to be ok and simple ignored when a non-existing file got tried. I would like a warning, anyway... |
ndeloof
pushed a commit
to ndeloof/compose
that referenced
this issue
Jan 30, 2023
ndeloof
pushed a commit
to ndeloof/compose
that referenced
this issue
Jan 30, 2023
ndeloof
pushed a commit
to ndeloof/compose
that referenced
this issue
Jan 31, 2023
ndeloof
pushed a commit
to ndeloof/compose
that referenced
this issue
Feb 1, 2023
ndeloof
pushed a commit
that referenced
this issue
Feb 2, 2023
milas
pushed a commit
to milas/compose
that referenced
this issue
Feb 24, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Our docker-compose.yml file references an
env_file
that gets auto-generated by a script that we run beforedocker-compose up
. Which means that the env file isn't available when we rundocker-compose build
. Since the build process doesn't use environment variables, this shouldn't be a problem. Butdocker-compose build
exits with an error of "Couldn't find env file: /foo/bar/my.env".Instead,
docker-compose build
should ignore the non-existent env file. It should only print the error if the env file isn't found by the timedocker-compose up
is run.The text was updated successfully, but these errors were encountered: