On Fri, Feb 03, 2023 at 03:35:45PM +0100, Gabriel Niebler wrote: > The function _idmapped_mount tries to change the ownership of the mountpoint > for which it aims to create an idmapped mount, to ensure that the mapped UID > and GID can actually create objects within it. Some tests set up a read-only > mount, however, which lets the chown call fail. This patch fixes the > function to check whether the mount is read-only and skip the chown, if so. > > Signed-off-by: Gabriel Niebler <gniebler@xxxxxxxx> > --- Looks good to me, Reviewed-by: Christian Brauner <brauner@xxxxxxxxxx> Thanks for fixing this!