-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add Tmpfs support to the python API #939
Conversation
Can someone explain why the unittests are failling? |
The expected output doesn't match the actual output. Look at |
Ah. A typo. Should be fixed now. |
Wtf, They ARE signed ... |
This adds support for the Tmpfs option introduced in Docker 1.10. See: moby/moby#13587 Signed-off-by: Jan Losinski <losinski@wh2.tu-dresden.de>
Signed-off-by: Jan Losinski <losinski@wh2.tu-dresden.de>
Signed-off-by: Jan Losinski <losinski@wh2.tu-dresden.de>
if not isinstance(tmpfs, list): | ||
raise ValueError( | ||
'Tmpfs spec must be a list' | ||
) |
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.
There seems to be some inconsistencies in the error messages here. This one says must be a list, but the previous branch accepted dicts. The next one says they must be something else.
Merged as part of #994 |
This adds support for the recently (1.10) added "--tmpfs" option in docker.
It is used to made private temporary filesystem mounts inside the container,
even if the container itself is not privileged to mount anything. Its a bit modeled
after the "binds" feature for volumes. The first commit provide the necessary
code changes. The second provides some unittests and the third adds
documentation.
You can see details about the tmpfs feature in the related docker PR:
moby/moby#13587