[RFC PATCH 00/16] gitlab-ci: Introduce "CI job maintainer" concept, mark jobs maintained

Philippe Mathieu-Daudé posted 16 patches 3 years, 5 months ago
Failed in applying to current master (apply log)
.gitlab-ci.d/containers.yml  | 138 +++++++++++++++++++++++---------
.gitlab-ci.d/crossbuilds.yml |  77 ++++++++++++------
.gitlab-ci.d/edk2.yml        |   4 +
.gitlab-ci.d/opensbi.yml     |   4 +
.gitlab-ci.yml               | 150 +++++++++++++++++++++++++----------
5 files changed, 274 insertions(+), 99 deletions(-)
[RFC PATCH 00/16] gitlab-ci: Introduce "CI job maintainer" concept, mark jobs maintained
Posted by Philippe Mathieu-Daudé 3 years, 5 months ago
Hi,

TL;DR: Start reviewing from patch 10.

To release the burden on the project and subsystem maintainers
we want to rely more on GitLab continuous integration jobs.

We'd like the subsystem maintainers to use GitLab more, but
we need to provide contact in case something gets wrong (a job
failed: is it a bug in the code? is the test buggy? is GitLab
working? ...).

To avoid project/subsystem maintainers to worry about that, we
introduce the concept of "CI job maintainer". If a job fails,
there is someone responsible for it, able to help to figure out
what went wrong, until having the CI job passing again.

CI "job maintainers" are listed along the YAML job, having their
name and email in environment variable. Eventually script will
consume this variables to notify the maintainer.

I don't plan this to get merged, I just shoot a RFC to see what
the community thinks about this problem.

Patches 1-9 are preparatory cleanup. The proof of concept is
introduced (patch 10). Jobs without maintainer listed aren't
run. Then we gradually add maintainers, jobs get run again,
until restoring the current mainstream state (same jobs ran).

The final patch disable Avocado tests, as they are flaky, and
we want maintainers become confident GitLab CI is helpful for
them, so we don't want flaky tests for now. We might restore
these later.

Regards,

Phil.

Based-on: <20201110121606.2792442-1-philmd@redhat.com>

Philippe Mathieu-Daudé (16):
  gitlab-ci: Replace YAML anchors by extends (cross_system_build_job)
  gitlab-ci: Replace YAML anchors by extends (native_build_job)
  gitlab-ci: Replace YAML anchors by extends (native_test_job)
  gitlab-ci: Replace YAML anchors by extends (acceptance_test_job)
  gitlab-ci: Replace YAML anchors by extends (container_job)
  gitlab-ci: Rename acceptance_test_job -> integration_test_job
  gitlab-ci: Extract common job definition as 'cross_common_job'
  gitlab-ci: Extract common job definition as 'native_common_job'
  gitlab-ci: Set default workflow rule
  gitlab-ci: Introduce the CI "job maintainer" concept
  gitlab-ci: Mark some jobs maintained by Red Hat Virt CI team
  gitlab-ci: Mark Bin Meng maintainer of the OpenSBI job
  gitlab-ci: Mark Alex Bennée maintainer of Debian / Ubuntu jobs
  gitlab-ci: Mark Stefan Weil maintainer of the TCI job
  gitlab-ci: Volunteer to maintain Debian based and hobbyist jobs
  gitlab-ci: Do not automatically run integration tests for push events

 .gitlab-ci.d/containers.yml  | 138 +++++++++++++++++++++++---------
 .gitlab-ci.d/crossbuilds.yml |  77 ++++++++++++------
 .gitlab-ci.d/edk2.yml        |   4 +
 .gitlab-ci.d/opensbi.yml     |   4 +
 .gitlab-ci.yml               | 150 +++++++++++++++++++++++++----------
 5 files changed, 274 insertions(+), 99 deletions(-)

-- 
2.26.2