On Wed, Nov 11, 2020 at 10:15 PM Nicolas Iooss <nicolas.iooss@xxxxxxx> wrote: > On Mon, Nov 2, 2020 at 12:12 PM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote: > > > > On Mon, Nov 2, 2020 at 11:17 AM Petr Lautrbach <plautrba@xxxxxxxxxx> wrote: > > > On Mon, Nov 02, 2020 at 10:43:10AM +0100, Ondrej Mosnacek wrote: > > > > The testsuite will soon be switching to testing multiple filesystems, > > > > which exposes a bug in F32 image's kernel. Since Fedora 33 has been > > > > released recently and the testsuite runs just fine on it, just bump the > > > > image version to avoid the bug. > > > > > > > > This commit also fixes the script to read out the Fedora image version > > > > from environment variables instead of using hard-coded values. > > > > > > > > Signed-off-by: Ondrej Mosnacek <omosnace@xxxxxxxxxx> > > > > > > Would it make sense to use Rawhide images from the link bellow? > > > > > > https://download.fedoraproject.org/pub/fedora/linux/development/rawhide/Cloud/x86_64/images/ > > > > > > I think Paul already asked for it in past but I'm not sure what was the outcome. > > > > It is possible and already being used for the testsuite CI. However, > > it is a bit harder to get to work reliably and there is always the > > possibility of rawhide being randomly broken. It would require some > > more extensive updates to the scripts (including a couple fixes from > > my pending patches for the testsuite CI), so IMHO it should be done as > > a separate patch (if at all). > > Hi, the patch looks good, but it does not fix everything: e2fsprogs, > jfsutils and dosfstools also need to be added to > scripts/ci/fedora-test-runner.sh, like what > https://github.com/SELinuxProject/selinux-testsuite/commit/d4e507f78a1784334611421cdfa3683a8214b22d > documented. Do you want to submit a patch about this? Yes, I forgot that those would also be needed... I just sent a patch to the list. -- Ondrej Mosnacek Software Engineer, Platform Security - SELinux kernel Red Hat, Inc.
