aufs: apply dirperm1 by default if supported - #11799
Conversation
|
cc @tiborvass Also it seems like enabling |
|
+1 for this, I like it. Ping @vbatts @unclejack feel free to step in to say if you don't like the way this is implemented. |
There was a problem hiding this comment.
should be
base, err := ioutil.TempDir("", "docker-aufs-base")
if err != nil {Same for union, err :=
|
@jfrazelle this would be a great fix that a lot of people would welcome. Also it would close an issue with a 3-digit number :P @dqminh would you mind removing the part that mentions the issue in some KNOWN issues section somewhere? |
Automatically detect support for aufs `dirperm1` option and apply it. `dirperm1` tells aufs to check the permission bits of the directory on the topmost branch and ignore the permission bits on all lower branches. It can be used to fix aufs' permission bug (i.e., upper layer having broader mask than the lower layer). More information about the bug can be found at moby#783 `dirperm1` man page is at: http://aufs.sourceforge.net/aufs3/man.html Signed-off-by: Daniel, Dao Quang Minh <dqminh89@gmail.com>
@tiborvass this won't fix it for system that doesnt have |
|
@dqminh fair enough! Thanks :) |
Since `dirperm1` requires a more recent aufs patch than many current OS release, we cant remove moby#783 completely. This documents that docker will apply `dirperm1` automatically for systems that support it Signed-off-by: Daniel, Dao Quang Minh <dqminh89@gmail.com>
|
LGTM |
It's easier for users to check if their systems support dirperm1 just by using docker info Signed-off-by: Daniel, Dao Quang Minh <dqminh89@gmail.com>
|
@tiborvass i added dirperm1 support status to |
|
@tiborvass is this merge-able for 1.6, or is it too late now ? |
|
I added the milestone @crosbymichael is that ok with you |
|
LGTM |
aufs: apply dirperm1 by default if supported
There was a problem hiding this comment.
While investigating some other (transient) docker problem, I saw this error (once) in my kernel log — is it a harmless error produced here while testing the presence of dirperm1? Google wasn't very helpful.
[ 16.825747] aufs au_opts_parse:1155:docker[1679]: unknown option dirperm1
There was a problem hiding this comment.
OK, I'm guessing yes:
# docker info | grep Dirperm1
Dirperm1 Supported: false
Automatically detect support for aufs
dirperm1option and apply it.dirperm1tells aufs to check the permission bits of the directory on thetopmost branch and ignore the permission bits on all lower branches.
It can be used to fix aufs' permission bug (i.e., upper layer having
broader mask than the lower layer).
dirperm1man page is at: http://aufs.sourceforge.net/aufs3/man.htmlFixes #783