Conversation
|
No, I don't think we should do this, and this definitely doesn't address docker/compose#745 (just shifts the problem from directory name or .env to the compose file) |
|
@thaJeztah then I misunderstood the requirements set on docker/compose#745 |
|
@thaJeztah do you mean because it doesn't persist the project name in a separate file from the docker-compose.yml file, and/or because compose does not save the auto-generated project name into said separate file? This solution does allow persistence of the project name independently of choices that may affect other (not compose related) parts of a project (folder name, checking in a .env file), so I think it's fair to say it covers at least some of the use cases -- I think some clarification on what it's missing would be helpful. |
|
@thaJeztah ok, reading more carefully your initial request docker/compose#745. IIUC you're expecting we store "user session state" on filesystem aside the compose.yaml file so that user only has to set That being said, comments on docker/compose#745 demonstrate we also have user who expect they can hard-code the project name in the compose file to make it fully deterministic. I think we can have both
|
|
+1 to have an option to define name as part of the compose file. Indeed, it does not need to be the only option, but in many cases defining it inside the compose file is the most intiuitive and natural solution. |
EricHripko
left a comment
There was a problem hiding this comment.
Tiny typo inline 💬 More general question: how do we expect it to work for cases when multiple Compose files are combined?
Like other elements : if defined in an override file, will override. |
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
0257a0d to
bb155bb
Compare
EricHripko
left a comment
There was a problem hiding this comment.
Looks good (with one typo/suggestion inline) ✅
EricHripko
left a comment
There was a problem hiding this comment.
Looks great ✅ Very excited about this addition 🎉
What this PR does / why we need it:
introduce top-level
nameto address docker/compose#745