docker / cli

The Docker CLI
Apache License 2.0
4.76k stars 1.89k forks source link

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

Open thaJeztah opened 6 years ago

thaJeztah commented 6 years ago

This is a follow-up to https://github.com/moby/moby/pull/35467#issuecomment-345036525

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

ping @kolyshkin @cpuguy83 @AkihiroSuda PTAL

cpuguy83 commented 6 years ago

How come --shm-size is affected here?

thaJeztah commented 6 years ago

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.