Re: [PATCH libdrm] CI: Capture test logs as GitLab artifacts

[Eric Engestrom <eric.engestrom@intel.com>]

On Thursday, 2018-09-06 11:01:17 +0100, Daniel Stone wrote:
> GitLab CI already captures all the stdout/stderr output from the build
> process as the log. However, some other important information is hidden
> in other log files.
> 
> Taken from Wayland, capture logs from the configuration process as well
> as from every check.
> 
> Signed-off-by: Daniel Stone <daniels@collabora.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@gmail.com>
> Cc: Lucas De Marchi <lucas.de.marchi@gmail.com>
> Cc: Eric Engeström <eric.engestrom@intel.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> ---
>  .gitlab-ci.yml | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index eee6abfc..50ec8527 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -1,6 +1,29 @@
> +.artifacts-meson: &artifacts-meson

Ooooh, variables? I might try to dedup the 'meson build' and
'autotools build' code with those :)

> +  when: always
> +  paths:
> +   - _build/meson-logs
> +
> +.artifacts-autotools: &artifacts-autotools
> +  when: always
> +  paths:
> +    - _build/*.log
> +    - _build/amdgpu/*.log
> +    - _build/etnaviv/*.log
> +    - _build/exynos/*.log
> +    - _build/freedreno/*.log
> +    - _build/intel/*.log
> +    - _build/libkms/*.log
> +    - _build/nouveau/*.log
> +    - _build/omap/*.log
> +    - _build/radeon/*.log
> +    - _build/tegra/*.log

All of the above can be simplified to:
- _build/*/*.log

(vc4 is missing btw)

With vc4 added, or /*/ used:
Acked-by: Eric Engestrom <eric.engestrom@intel.com>

> +    - _build/tests/*.log
> +    - _build/tests/*/*.log
> +
>  latest-meson:
>    stage: build
>    image: base/archlinux:latest
> +  artifacts: *artifacts-meson
>    before_script:
>      - pacman -Syu --noconfirm --needed
>          base-devel
> @@ -35,6 +58,7 @@ latest-meson:
>  latest-autotools:
>    stage: build
>    image: base/archlinux:latest
> +  artifacts: *artifacts-autotools
>    before_script:
>      - pacman -Syu --noconfirm --needed
>          base-devel
> @@ -69,6 +93,7 @@ latest-autotools:
>  oldest-meson:
>    stage: build
>    image: debian:stable
> +  artifacts: *artifacts-meson
>    before_script:
>      - printf > /etc/dpkg/dpkg.cfg.d/99-exclude-cruft "%s\n"
>          'path-exclude=/usr/share/doc/*'
> @@ -125,6 +150,7 @@ oldest-meson:
>  oldest-autotools:
>    stage: build
>    image: debian:stable
> +  artifacts: *artifacts-autotools
>    before_script:
>      - printf > /etc/dpkg/dpkg.cfg.d/99-exclude-cruft "%s\n"
>          'path-exclude=/usr/share/doc/*'
> -- 
> 2.19.0.rc0
> 
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel