On 14/06/2021 20:29, Heinrich Schuchardt wrote: > On 6/13/21 4:07 PM, Alper Nebi Yasak wrote: >> >> supermin: failed to find a suitable kernel (host_cpu=x86_64). >> >> I looked for kernels in /boot and modules in /lib/modules. >> >> [...] >> >> diff --git a/tools/docker/Dockerfile b/tools/docker/Dockerfile >> index d2f0074ee8a6..563b16639e54 100644 >> --- a/tools/docker/Dockerfile >> +++ b/tools/docker/Dockerfile >> @@ -71,6 +71,7 @@ RUN apt-get update && apt-get install -y \ >> libssl-dev \ >> libudev-dev \ >> libusb-1.0-0-dev \ >> + linux-image-kvm \ >> lzma-alone \ >> lzop \ >> mount \ >> @@ -99,6 +100,9 @@ RUN apt-get update && apt-get install -y \ >> zip \ >> && rm -rf /var/lib/apt/lists/* >> >> +# Make kernels readable for libguestfs tools to work correctly >> +RUN chmod +r /boot/vmlinu* /lib/modules/*/vmlinu* || true > > /lib/modules/4.15.0-1092-kvm# find . -name 'vm*' > does not find any file. > > We don't need '|| true' if you do not refer to non-existent files.
I included both dirs as libguestfs checks both of them. If that doesn't convince you, I can send a v3 with just `chmod +r /boot/vmlinu*`. I actually misread the error message above as mentioning both dirs for kernel so I didn't explicitly write it in the commit message, but the source code [1] confirms that both are checked. [1] https://github.com/libguestfs/supermin/blob/master/src/format_ext2_kernel.ml#L28-L49 (sorry for the late reply)