Re: overlayfs regression

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jun 24, 2020 at 06:25:55PM +0300, Amir Goldstein wrote:
> On Wed, Jun 24, 2020 at 5:48 PM Christian Brauner
> <christian.brauner@xxxxxxxxxx> wrote:
> >
> > Hey Miklosz,
> > hey Amir,
> >
> > We've been observing regressions in our containers test-suite with
> > commit:
> >
> > Author: Miklos Szeredi <mszeredi@xxxxxxxxxx>
> > Date:   Tue Mar 17 15:04:22 2020 +0100
> >
> >     ovl: separate detection of remote upper layer from stacked overlay
> >
> >     Following patch will allow remote as upper layer, but not overlay stacked
> >     on upper layer.  Separate the two concepts.
> >
> >     This patch is doesn't change behavior.
> >
> >     Signed-off-by: Miklos Szeredi <mszeredi@xxxxxxxxxx>
> >
> 
> Are you sure this is the offending commit?
> Look at it. It is really moving a bit of code around and should not
> change logic.
> There are several other commits in 5.7 that could have gone wrong...

Yeah, most likely. I can do a bisect but it might take a little until I
get around to it. Is that ok?

> 
> > It suddenly consistently reports:
> > [2422.695340] overlayfs: filesystem on '/home/lxcunpriv/.local/share/lxc/c2/overlay/delta' not supported as upperdir
> > in dmesg where it used to work fine for basically 6 years when we added
> > that test. The test creates a container c2 that uses the rootfs of
> > another container c1 (normal directory on an ext4 filesystem). Here you
> > can see the full mount options:
> >
> > Invalid argument - Failed to mount "/home/lxcunpriv/.local/share/lxc/c1/rootfs" on "/usr/lib/x86_64-linux-gnu/lxc" with options "upperdir=/home/lxcunpriv/.local/share/lxc/c2/overlay/delta,lowerdir=/home/lxcunpriv/.local/share/lxc/c1/rootfs,workdir=/home/lxcunpriv/.local/share/lxc/c2/overlay/work"
> >
> 
> /home/lxcunpriv/.local/share/lxc/c2/overlay/delta' is ext4?

Yeah, so what we do is:

lower -> /foo/c1/rootfs

then we create the directory for the new container:

mkdir_p(/foo/c2)
mkdir_p(/foo/c2/rootfs)
mkdir_p(/foo/c2/overlay)
mkdir_p(/foo/c2/overlay/delta)
mkdir_p(/foo/c2/overlay/workdir)

ensuring that delta + work are on the same mount and that workdir is
empty. The we perform the mount you see above.

Christian



[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux