Skip to content

Conversation

ndeloof
Copy link
Collaborator

@ndeloof ndeloof commented Apr 4, 2023

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Copy link
Collaborator

@glours glours left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good, we should just add a test case for volumes_from with a service name

default: null
volumes_from:
- zot
- container:xxx
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- container:xxx
- service:yyy"
- container:xxx

Copy link
Member

@milas milas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙈

@glours glours merged commit 1af0f42 into compose-spec:master Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] volumes_from external container fails: service xxx depends on undefined service container: invalid compose project
3 participants