Skip to content
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

[WIP] Add support for build time volumes #5736

Closed
wants to merge 1 commit into from

Conversation

ymc-dabe
Copy link

@ymc-dabe ymc-dabe commented Mar 1, 2018

Resolves #4423

Signed-off-by: Daniel Beyer ymc-dabe@vcs.ymc.ch

Resolves docker#4423

Signed-off-by: Daniel Beyer <ymc-dabe@vcs.ymc.ch>
@ymc-dabe
Copy link
Author

ymc-dabe commented Mar 1, 2018

Please note this this should not be merged so far, since docker-py is lacking support of build time volumes, see:

Also please do not forget to update requirements.txt accordingly (it must not reference github/ymc-dabe/compose.

@ijc
Copy link

ijc commented May 20, 2019

Since this is still WIP >1yr later and both the prerequisites have both been closed (referencing docker/docker-py#1937 (comment)) I think we should close this, so that's what I'm going to do.

Thanks for your contribution even if it didn't work out.

WRT the link to moby/buildkit#442 (in the comment linked above) I think #6584 is relevant in that context.

@ijc ijc closed this May 20, 2019
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.

Add support for build volumes
2 participants