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

Emil Velikov emil.l.velikov at gmail.com
Thu Sep 6 13:44:02 UTC 2018


On 6 September 2018 at 12:02, Eric Engestrom <eric.engestrom at intel.com> wrote:
> 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 at collabora.com>
>> Cc: Rodrigo Vivi <rodrigo.vivi at gmail.com>
>> Cc: Lucas De Marchi <lucas.de.marchi at gmail.com>
>> Cc: Eric Engeström <eric.engestrom at intel.com>
>> Cc: Daniel Vetter <daniel.vetter at 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 at intel.com>
>
I would fold everything in these tree lines.
- _build/*.log
- _build/*/*.log
- _build/*/*/*.log

It covers everything existing, plus catches future ones ;-)
With that
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

-Emil


More information about the dri-devel mailing list