-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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 env file resolution #1099
Fix env file resolution #1099
Conversation
if k in override: | ||
d[k] = override[k] | ||
|
||
return d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this (merge_service_dicts
) isn't used yet. I guess this was pulled in from the other branch?
I like it What about host volume binds with relative paths? Do you plan on addressing those in a separate branch? It seems like volume paths should probably also be relative to the |
Yes, good point. I'll update it so volume paths are treated similarly. (That's going to be fun, because we expand |
Signed-off-by: Aanand Prasad <aanand.prasad@gmail.com>
Signed-off-by: Aanand Prasad <aanand.prasad@gmail.com>
8f26b7b
to
528bed9
Compare
After some thought, I'm going to do volume paths separately. This is rebased and ready to go. |
LGTM |
Fix env file resolution Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
Environment variable file paths should be considered relative to the compose file, not to the current directory. These are usually the same, unless the user specifies an alternate one in a different directory with
-f
.