Re: [PATCH v2 8/9] tests/docker: Add dockerfile for Alpine Linux

[Daniel P . Berrangé]

On Mon, Jan 18, 2021 at 02:38:07PM +0800, Jiaxun Yang wrote:
> Alpine Linux[1] is a security-oriented, lightweight Linux distribution
> based on musl libc and busybox.
> 
> It it popular among Docker guests and embedded applications.
> 
> Adding it to test against different libc.
> 
> [1]: https://alpinelinux.org/
> 
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
>  tests/docker/dockerfiles/alpine.docker | 57 ++++++++++++++++++++++++++
>  1 file changed, 57 insertions(+)
>  create mode 100644 tests/docker/dockerfiles/alpine.docker
> 
> diff --git a/tests/docker/dockerfiles/alpine.docker 
> b/tests/docker/dockerfiles/alpine.docker
> new file mode 100644
> index 0000000000..5be5198d00
> --- /dev/null
> +++ b/tests/docker/dockerfiles/alpine.docker
> @@ -0,0 +1,57 @@
> +
> +FROM alpine:edge
> +
> +RUN apk update
> +RUN apk upgrade
> +
> +# Please keep this list sorted alphabetically
> +ENV PACKAGES \
> +     alsa-lib-dev \
> +     bash \
> +     bison \

This shouldn't be required.

> +     build-base \

This seems to be a meta packae that pulls in other
misc toolchain packages. Please list the pieces we
need explicitly instead.

> +     coreutils \
> +     curl-dev \
> +     flex \

This shouldn't be needed.

> +     git \
> +     glib-dev \
> +     glib-static \
> +     gnutls-dev \
> +     gtk+3.0-dev \
> +     libaio-dev \
> +     libcap-dev \

Should not be required, as we use cap-ng.

> +     libcap-ng-dev \
> +     libjpeg-turbo-dev \
> +     libnfs-dev \
> +     libpng-dev \
> +     libseccomp-dev \
> +     libssh-dev \
> +     libusb-dev \
> +     libxml2-dev \
> +     linux-headers \

Is this really needed ? We don't install kernel-headers on other
distros AFAICT.

> +     lzo-dev \
> +     mesa-dev \
> +     mesa-egl \
> +     mesa-gbm \
> +     meson \
> +     ncurses-dev \
> +     ninja \
> +     paxmark \

What is this needed for ?

> +     perl \
> +     pulseaudio-dev \
> +     python3 \
> +     py3-sphinx \
> +     shadow \

Is this really needed ?

> +     snappy-dev \
> +     spice-dev \
> +     texinfo \
> +     usbredir-dev \
> +     util-linux-dev \
> +     vde2-dev \
> +     virglrenderer-dev \
> +     vte3-dev \
> +     xfsprogs-dev \
> +     zlib-dev \
> +     zlib-static
> +
> +RUN apk add $PACKAGES

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|