[libvirt PATCH 0/4] Functional CI - GitLab enablement

Erik Skultety posted 4 patches 2 years, 1 month ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1647339160.git.eskultet@redhat.com
.gitlab-ci.yml     |  99 +-------
ci/gitlab.yml      | 606 ++-------------------------------------------
ci/integration.yml |  99 ++++++++
ci/manifest.yml    |  21 +-
4 files changed, 142 insertions(+), 683 deletions(-)
create mode 100644 ci/integration.yml
[libvirt PATCH 0/4] Functional CI - GitLab enablement
Posted by Erik Skultety 2 years, 1 month ago
RFC here [1]

Since RFC:
    - renamed the stage to integration_tests
    - dropped both the test child pipeline as well as triggering a
    multi-project CI pipeline to build the Perl bindings and instead use the
    latest bindings artifacts we have available from the libvirt-perl project.
        -> basically now we only have a regular CI stage with no extra jobs
           that just spins up a bunch of private runners and uploads RPM
           artifacts into them and runs the TCK tests
    - replaced sed invocations to edit daemon logging settings with augtool
    - used a pattern for log filters from [2]
    - used the "$CI_JOB_STATUS" variable in the after_script to determine
      whether the main script failed so as not to try moving and collecting
      logs on successful jobs (the logs would have been published as artifacts
                               on failures anyway)

Demo pipeline: https://gitlab.com/eskultety/libvirt/-/pipelines/491838828

[1] https://listman.redhat.com/archives/libvir-list/2022-January/227947.html
[2] https://libvirt.org/kbase/debuglogs.html#less-verbose-logging-for-qemu-vms

Erik Skultety (4):
  ci: gitlab: Refresh gitlab.yml
  ci: manifest: Publish RPMs as artifacts on CentOS Stream and Fedoras
  gitlab-ci: Introduce new 'integration_tests' pipeline stage
  DO NOT MERGE: Demo a functional CI pipeline running Fedoras CentOS
    only

 .gitlab-ci.yml     |  99 +-------
 ci/gitlab.yml      | 606 ++-------------------------------------------
 ci/integration.yml |  99 ++++++++
 ci/manifest.yml    |  21 +-
 4 files changed, 142 insertions(+), 683 deletions(-)
 create mode 100644 ci/integration.yml

-- 
2.34.1

Re: [libvirt PATCH 0/4] Functional CI - GitLab enablement
Posted by Michal Prívozník 2 years, 1 month ago
On 3/15/22 11:14, Erik Skultety wrote:
> RFC here [1]
> 
> Since RFC:
>     - renamed the stage to integration_tests
>     - dropped both the test child pipeline as well as triggering a
>     multi-project CI pipeline to build the Perl bindings and instead use the
>     latest bindings artifacts we have available from the libvirt-perl project.
>         -> basically now we only have a regular CI stage with no extra jobs
>            that just spins up a bunch of private runners and uploads RPM
>            artifacts into them and runs the TCK tests
>     - replaced sed invocations to edit daemon logging settings with augtool
>     - used a pattern for log filters from [2]
>     - used the "$CI_JOB_STATUS" variable in the after_script to determine
>       whether the main script failed so as not to try moving and collecting
>       logs on successful jobs (the logs would have been published as artifacts
>                                on failures anyway)
> 
> Demo pipeline: https://gitlab.com/eskultety/libvirt/-/pipelines/491838828
> 
> [1] https://listman.redhat.com/archives/libvir-list/2022-January/227947.html
> [2] https://libvirt.org/kbase/debuglogs.html#less-verbose-logging-for-qemu-vms
> 
> Erik Skultety (4):
>   ci: gitlab: Refresh gitlab.yml
>   ci: manifest: Publish RPMs as artifacts on CentOS Stream and Fedoras
>   gitlab-ci: Introduce new 'integration_tests' pipeline stage
>   DO NOT MERGE: Demo a functional CI pipeline running Fedoras CentOS
>     only
> 
>  .gitlab-ci.yml     |  99 +-------
>  ci/gitlab.yml      | 606 ++-------------------------------------------
>  ci/integration.yml |  99 ++++++++
>  ci/manifest.yml    |  21 +-
>  4 files changed, 142 insertions(+), 683 deletions(-)
>  create mode 100644 ci/integration.yml
> 

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

Michal