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

Docker CLI should use mount API for --tmpfs #710

Open
thaJeztah opened this issue Nov 22, 2017 · 2 comments
Open

Docker CLI should use mount API for --tmpfs #710

thaJeztah opened this issue Nov 22, 2017 · 2 comments

Comments

@thaJeztah
Copy link
Member

thaJeztah commented Nov 22, 2017

This is a follow-up to moby/moby#35467 (comment)

The --tmpfs option adds tmpfs mounts to a container. This option is currently handled daemon-side, where mounts specified through --tmpfs are merged with the default mounts, mounts from -v / --volume, and mounts specified through --mount.

We should change the CLI to use the Mount API

Question

  • Should we do the same for --shm-size?
  • If we do; where are the defaults set? i.e., we likely don't want the CLI to be aware of any defaults set by the daemon for /dev/shm; how can we provide a custom --shm-size, but keep other defaults as-is?

ping @kolyshkin @cpuguy83 @AkihiroSuda PTAL

@cpuguy83
Copy link
Collaborator

How come --shm-size is affected here?

@thaJeztah
Copy link
Member Author

That option sets the size for the default /dev/shm mount; Also wondering if the "use a custom --mount type=tmpfs,target=/dev/shm... approach should be the canonical way to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants