Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section

[Philippe Mathieu-Daudé]

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 <thuth@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.bennee@linaro.org>
>>> -M: Fam Zheng <fam@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.