[libvirt PATCH 0/4] gitlab: improve CI speed

Daniel P. Berrangé posted 4 patches 2 years, 11 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210510190801.945694-1-berrange@redhat.com
.gitlab-ci.yml      | 31 ++++++++++++++-----------------
ci/cirrus/build.yml |  6 ++++--
2 files changed, 18 insertions(+), 19 deletions(-)
[libvirt PATCH 0/4] gitlab: improve CI speed
Posted by Daniel P. Berrangé 2 years, 11 months ago
In several of the jobs we are running the libvirt build twice.

We also uncessarily run the syntax-check job multiple times
which causes more work for contributors looking at failed
jobs.

Daniel P. Berrangé (4):
  gitlab: only print failed tests in codestyle job
  gitlab: remove ability to skip 'ninja dist'
  gitlab: run "meson" directly instead of "ninja" indirection
  gitlab: avoid building libvirt twice and running syntax-check twice

 .gitlab-ci.yml      | 31 ++++++++++++++-----------------
 ci/cirrus/build.yml |  6 ++++--
 2 files changed, 18 insertions(+), 19 deletions(-)

-- 
2.31.1


Re: [libvirt PATCH 0/4] gitlab: improve CI speed
Posted by Michal Prívozník 2 years, 11 months ago
On 5/10/21 9:07 PM, Daniel P. Berrangé wrote:
> In several of the jobs we are running the libvirt build twice.
> 
> We also uncessarily run the syntax-check job multiple times
> which causes more work for contributors looking at failed
> jobs.
> 
> Daniel P. Berrangé (4):
>   gitlab: only print failed tests in codestyle job
>   gitlab: remove ability to skip 'ninja dist'
>   gitlab: run "meson" directly instead of "ninja" indirection
>   gitlab: avoid building libvirt twice and running syntax-check twice
> 
>  .gitlab-ci.yml      | 31 ++++++++++++++-----------------
>  ci/cirrus/build.yml |  6 ++++--
>  2 files changed, 18 insertions(+), 19 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal