Uh oh!
There was an error while loading. Please reload this page.
Add masked and readonly paths - #364
Conversation
| // RootfsPropagation is the rootfs mount propagation mode for the container. | ||
| RootfsPropagation string `json:"rootfsPropagation,omitempty"` | ||
| // MaskedPaths masks over the provided paths inside the container. | ||
| MaskedPaths []string `json:"maskedPaths"` |
There was a problem hiding this comment.
nit: Add omitempty here and below
vishh
commented
Apr 1, 2016
LGTM |
| ## Readonly Paths | ||
| readonlyPaths will set the provieded paths as readonly inside the container. |
There was a problem hiding this comment.
“provieded” → “provided”. And probably readonlyPaths (and similar for your earlier maskedPaths docs) to match the rest of the spec.
wking
commented
Apr 1, 2016
On Thu, Mar 31, 2016 at 05:15:51PM -0700, Michael Crosby wrote:
I'm still not sure we need these 1, but if they're going in, should |
Fixesopencontainers#320 This adds the maskedPaths and readonlyPaths fields to the spec so that proper masking and setting of files in /proc can be configured. Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
crosbymichael
commented
Apr 1, 2016
@vbatts i tried to updated the scheme stuff, let me know if I did it wrong |
mrunalp
commented
Apr 1, 2016
The schema changes look good. (I also tested them using the validate tool). |
Fixes#320
This adds the maskedPaths and readonlyPaths fields to the spec so that
proper masking and setting of files in /proc can be configured.
Signed-off-by: Michael Crosby crosbymichael@gmail.com