Closed Harjot1Singh closed 4 years ago
@Harjot1Singh haven't considered that case - would be a nice improvement! I'll try to update this once I have time. In the mean time - PRs welcome as usual :)
Relevant bit of code: https://github.com/exoframejs/exoframe-server/blob/master/src/docker/templates/compose.js#L45-L54
Done ;)
Description
When you specify label arguments as a list in
docker-compose.yml
, you end up with them being cast to object notation incorrectly. I logged the transformeddocker.compose.yaml
in thedeployments
folder to confirm this:Minimal Test Case
Input:
Output:
Expected Output
You can see that the input labels are instead mapped to array indices, and it instead should be anything before the first
=
.Proposed Solution
When mapping docker-compose labels that are supplied as an array, first parse them into an object. Something like: