On 3/8/21 6:01 PM, Thomas Huth wrote: > On 08/03/2021 17.35, Philippe Mathieu-Daudé wrote: >> On 3/8/21 4:46 PM, Thomas Huth wrote: >>> The status of the gitlab-CI files is currently somewhat confusing, and >>> it is often not quite clear whether a patch should go via my tree or >>> via the testing tree of Alex. That situation has grown historically... >>> Initially, I was the only one using the gitlab-CI, just for my private >>> repository there. But in the course of time, the gitlab-CI switched to >>> use the containers from tests/docker/ (which is not part of the >>> gitlab-CI >>> section in the MAINTAINERS file), and QEMU now even switched to >>> gitlab.com >>> completely for the repository and will soon use it as its gating CI, >>> too, >>> so it makes way more sense if the gitlab-ci.yml files belong to the >>> people >>> who are owning the qemu-project on gitlab.com and take care of the >>> gitlab >>> CI there. Thus let's merge the gitlab-ci section into the common >>> "test and >>> build automation" section. >>> >>> While we're at it, I'm also removing the line with Fam there for now, >>> since he was hardly active during the last years in this area anymore. >>> If he ever gets more time for this part again in the future, we surely >>> can add the line back again. >>> >>> Now to avoid that Alex is listed here alone, Philippe and I agreed to >>> help as backup maintainers here, too. >>> >>> Signed-off-by: Thomas Huth <th...@redhat.com> >>> --- >>> v2: Keep Philippe and myself as maintainer instead of reviewer >>> >>> MAINTAINERS | 21 +++++++-------------- >>> 1 file changed, 7 insertions(+), 14 deletions(-) >>> >>> diff --git a/MAINTAINERS b/MAINTAINERS >>> index 26c9454823..5c4c179abb 100644 >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -3262,17 +3262,21 @@ F: include/hw/remote/iohub.h >>> Build and test automation >>> ------------------------- >>> -Build and test automation >>> +Build and test automation, Linux Continuous Integration >> >> I'd rather not limit it to Linux hosts... >> >>> M: Alex Bennée <alex.ben...@linaro.org> >>> -M: Fam Zheng <f...@euphon.net> >> >> I'm not sure about Fam removal, he is the one maintaining patchew >> (see [*] below)... Maybe we need an Ack from Paolo? > > We could split the Patchew part into a separate section... OTOH, there > are no patchew related files in the main git repo anyway that we track > in the MAINTAINERS file, so maybe it makes more sense to simply remove > that line? (Patchew admins are already mentioned here: > https://wiki.qemu.org/AdminContacts )
Also here: https://wiki.qemu.org/Testing/CI So it makes sense to me to remove the patchew line.
