* [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
@ 2021-03-08 15:46 Thomas Huth
2021-03-08 16:35 ` Philippe Mathieu-Daudé
` (4 more replies)
0 siblings, 5 replies; 9+ messages in thread
From: Thomas Huth @ 2021-03-08 15:46 UTC (permalink / raw
To: Alex Bennée, qemu-devel, Philippe Mathieu-Daudé
Cc: Peter Maydell, Willian Rampazzo, Wainer dos Santos Moschetta
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
M: Alex Bennée <alex.bennee@linaro.org>
-M: Fam Zheng <fam@euphon.net>
-R: Philippe Mathieu-Daudé <philmd@redhat.com>
+M: Philippe Mathieu-Daudé <philmd@redhat.com>
+M: Thomas Huth <thuth@redhat.com>
+R: Wainer dos Santos Moschetta <wainersm@redhat.com>
S: Maintained
F: .github/lockdown.yml
+F: .gitlab-ci.yml
+F: .gitlab-ci.d/
F: .travis.yml
F: scripts/ci/
F: tests/docker/
F: tests/vm/
F: scripts/archive-source.sh
+W: https://gitlab.com/qemu-project/qemu/pipelines
W: https://travis-ci.org/qemu/qemu
W: http://patchew.org/QEMU/
@@ -3289,17 +3293,6 @@ S: Maintained
F: .cirrus.yml
W: https://cirrus-ci.com/github/qemu/qemu
-GitLab Continuous Integration
-M: Thomas Huth <thuth@redhat.com>
-M: Philippe Mathieu-Daudé <philmd@redhat.com>
-M: Alex Bennée <alex.bennee@linaro.org>
-R: Wainer dos Santos Moschetta <wainersm@redhat.com>
-S: Maintained
-F: .gitlab-ci.yml
-F: .gitlab-ci.d/crossbuilds.yml
-F: .gitlab-ci.d/*py
-F: scripts/ci/gitlab-pipeline-status
-
Guest Test Compilation Support
M: Alex Bennée <alex.bennee@linaro.org>
R: Philippe Mathieu-Daudé <f4bug@amsat.org>
--
2.27.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 15:46 [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section Thomas Huth
@ 2021-03-08 16:35 ` Philippe Mathieu-Daudé
2021-03-08 17:01 ` Thomas Huth
2021-03-08 17:29 ` Wainer dos Santos Moschetta
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-03-08 16:35 UTC (permalink / raw
To: Thomas Huth, Alex Bennée, qemu-devel
Cc: Peter Maydell, Wainer dos Santos Moschetta, Willian Rampazzo,
Paolo Bonzini
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?
> -R: Philippe Mathieu-Daudé <philmd@redhat.com>
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
Erm I don't have green light at work, so please use my personal email:
M: Philippe Mathieu-Daudé <f4bug@amsat.org>
> +M: Thomas Huth <thuth@redhat.com>
> +R: Wainer dos Santos Moschetta <wainersm@redhat.com>
> S: Maintained
> F: .github/lockdown.yml
> +F: .gitlab-ci.yml
> +F: .gitlab-ci.d/
> F: .travis.yml
> F: scripts/ci/
> F: tests/docker/
> F: tests/vm/
> F: scripts/archive-source.sh
> +W: https://gitlab.com/qemu-project/qemu/pipelines
> W: https://travis-ci.org/qemu/qemu
> W: http://patchew.org/QEMU/
[*]
> @@ -3289,17 +3293,6 @@ S: Maintained
> F: .cirrus.yml
> W: https://cirrus-ci.com/github/qemu/qemu
>
> -GitLab Continuous Integration
> -M: Thomas Huth <thuth@redhat.com>
> -M: Philippe Mathieu-Daudé <philmd@redhat.com>
> -M: Alex Bennée <alex.bennee@linaro.org>
> -R: Wainer dos Santos Moschetta <wainersm@redhat.com>
> -S: Maintained
> -F: .gitlab-ci.yml
> -F: .gitlab-ci.d/crossbuilds.yml
> -F: .gitlab-ci.d/*py
> -F: scripts/ci/gitlab-pipeline-status
> -
> Guest Test Compilation Support
> M: Alex Bennée <alex.bennee@linaro.org>
> R: Philippe Mathieu-Daudé <f4bug@amsat.org>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 16:35 ` Philippe Mathieu-Daudé
@ 2021-03-08 17:01 ` Thomas Huth
2021-03-08 17:52 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 9+ messages in thread
From: Thomas Huth @ 2021-03-08 17:01 UTC (permalink / raw
To: Philippe Mathieu-Daudé, Alex Bennée, qemu-devel
Cc: Peter Maydell, Wainer dos Santos Moschetta, Willian Rampazzo,
Paolo Bonzini
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 )
Thomas
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 15:46 [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section Thomas Huth
2021-03-08 16:35 ` Philippe Mathieu-Daudé
@ 2021-03-08 17:29 ` Wainer dos Santos Moschetta
2021-03-08 17:49 ` Willian Rampazzo
` (2 subsequent siblings)
4 siblings, 0 replies; 9+ messages in thread
From: Wainer dos Santos Moschetta @ 2021-03-08 17:29 UTC (permalink / raw
To: Thomas Huth, Alex Bennée, qemu-devel,
Philippe Mathieu-Daudé
Cc: Peter Maydell, Willian Rampazzo
On 3/8/21 12: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(-)
Acked-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
>
> 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
> M: Alex Bennée <alex.bennee@linaro.org>
> -M: Fam Zheng <fam@euphon.net>
> -R: Philippe Mathieu-Daudé <philmd@redhat.com>
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> +M: Thomas Huth <thuth@redhat.com>
> +R: Wainer dos Santos Moschetta <wainersm@redhat.com>
> S: Maintained
> F: .github/lockdown.yml
> +F: .gitlab-ci.yml
> +F: .gitlab-ci.d/
> F: .travis.yml
> F: scripts/ci/
> F: tests/docker/
> F: tests/vm/
> F: scripts/archive-source.sh
> +W: https://gitlab.com/qemu-project/qemu/pipelines
> W: https://travis-ci.org/qemu/qemu
> W: http://patchew.org/QEMU/
>
> @@ -3289,17 +3293,6 @@ S: Maintained
> F: .cirrus.yml
> W: https://cirrus-ci.com/github/qemu/qemu
>
> -GitLab Continuous Integration
> -M: Thomas Huth <thuth@redhat.com>
> -M: Philippe Mathieu-Daudé <philmd@redhat.com>
> -M: Alex Bennée <alex.bennee@linaro.org>
> -R: Wainer dos Santos Moschetta <wainersm@redhat.com>
> -S: Maintained
> -F: .gitlab-ci.yml
> -F: .gitlab-ci.d/crossbuilds.yml
> -F: .gitlab-ci.d/*py
> -F: scripts/ci/gitlab-pipeline-status
> -
> Guest Test Compilation Support
> M: Alex Bennée <alex.bennee@linaro.org>
> R: Philippe Mathieu-Daudé <f4bug@amsat.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 15:46 [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section Thomas Huth
2021-03-08 16:35 ` Philippe Mathieu-Daudé
2021-03-08 17:29 ` Wainer dos Santos Moschetta
@ 2021-03-08 17:49 ` Willian Rampazzo
2021-03-08 19:12 ` Philippe Mathieu-Daudé
2021-03-09 12:45 ` Alex Bennée
2021-03-10 17:01 ` Fam Zheng
4 siblings, 1 reply; 9+ messages in thread
From: Willian Rampazzo @ 2021-03-08 17:49 UTC (permalink / raw
To: Thomas Huth
Cc: Peter Maydell, Alex Bennée, qemu-devel,
Wainer dos Santos Moschetta, Philippe Mathieu-Daudé
On Mon, Mar 8, 2021 at 12:46 PM Thomas Huth <thuth@redhat.com> 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.
Makes sense to me.
Acked-by: Willian Rampazzo <willianr@redhat.com>
>
> 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
> M: Alex Bennée <alex.bennee@linaro.org>
> -M: Fam Zheng <fam@euphon.net>
> -R: Philippe Mathieu-Daudé <philmd@redhat.com>
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> +M: Thomas Huth <thuth@redhat.com>
> +R: Wainer dos Santos Moschetta <wainersm@redhat.com>
Feel free to add me as a Reviewer: Willian Rampazzo <willianr@redhat.com>
> S: Maintained
> F: .github/lockdown.yml
> +F: .gitlab-ci.yml
> +F: .gitlab-ci.d/
> F: .travis.yml
> F: scripts/ci/
> F: tests/docker/
> F: tests/vm/
> F: scripts/archive-source.sh
> +W: https://gitlab.com/qemu-project/qemu/pipelines
> W: https://travis-ci.org/qemu/qemu
> W: http://patchew.org/QEMU/
>
> @@ -3289,17 +3293,6 @@ S: Maintained
> F: .cirrus.yml
> W: https://cirrus-ci.com/github/qemu/qemu
>
> -GitLab Continuous Integration
> -M: Thomas Huth <thuth@redhat.com>
> -M: Philippe Mathieu-Daudé <philmd@redhat.com>
> -M: Alex Bennée <alex.bennee@linaro.org>
> -R: Wainer dos Santos Moschetta <wainersm@redhat.com>
> -S: Maintained
> -F: .gitlab-ci.yml
> -F: .gitlab-ci.d/crossbuilds.yml
> -F: .gitlab-ci.d/*py
> -F: scripts/ci/gitlab-pipeline-status
> -
> Guest Test Compilation Support
> M: Alex Bennée <alex.bennee@linaro.org>
> R: Philippe Mathieu-Daudé <f4bug@amsat.org>
> --
> 2.27.0
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 17:01 ` Thomas Huth
@ 2021-03-08 17:52 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-03-08 17:52 UTC (permalink / raw
To: Thomas Huth, Alex Bennée, qemu-devel
Cc: Peter Maydell, Willian Rampazzo, Wainer dos Santos Moschetta,
Paolo Bonzini
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.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 17:49 ` Willian Rampazzo
@ 2021-03-08 19:12 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-03-08 19:12 UTC (permalink / raw
To: Willian Rampazzo, Thomas Huth
Cc: Peter Maydell, Alex Bennée, qemu-devel,
Wainer dos Santos Moschetta
On 3/8/21 6:49 PM, Willian Rampazzo wrote:
> On Mon, Mar 8, 2021 at 12:46 PM Thomas Huth <thuth@redhat.com> 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.
>
> Makes sense to me.
>
> Acked-by: Willian Rampazzo <willianr@redhat.com>
>
>>
>> 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
>> M: Alex Bennée <alex.bennee@linaro.org>
>> -M: Fam Zheng <fam@euphon.net>
>> -R: Philippe Mathieu-Daudé <philmd@redhat.com>
>> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
>> +M: Thomas Huth <thuth@redhat.com>
>> +R: Wainer dos Santos Moschetta <wainersm@redhat.com>
>
> Feel free to add me as a Reviewer: Willian Rampazzo <willianr@redhat.com>
Thank you Willian! :)
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 15:46 [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section Thomas Huth
` (2 preceding siblings ...)
2021-03-08 17:49 ` Willian Rampazzo
@ 2021-03-09 12:45 ` Alex Bennée
2021-03-10 17:01 ` Fam Zheng
4 siblings, 0 replies; 9+ messages in thread
From: Alex Bennée @ 2021-03-09 12:45 UTC (permalink / raw
To: Thomas Huth
Cc: Peter Maydell, Willian Rampazzo, qemu-devel,
Wainer dos Santos Moschetta, Philippe Mathieu-Daudé
Thomas Huth <thuth@redhat.com> writes:
> 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>
Pending changes requested by the others:
Acked-by: Alex Bennée <alex.bennee@linaro.org>
--
Alex Bennée
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
2021-03-08 15:46 [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section Thomas Huth
` (3 preceding siblings ...)
2021-03-09 12:45 ` Alex Bennée
@ 2021-03-10 17:01 ` Fam Zheng
4 siblings, 0 replies; 9+ messages in thread
From: Fam Zheng @ 2021-03-10 17:01 UTC (permalink / raw
To: Thomas Huth
Cc: Peter Maydell, qemu-devel, Wainer dos Santos Moschetta,
Philippe Mathieu-Daudé, Willian Rampazzo, Alex Bennée
[-- Attachment #1: Type: text/plain, Size: 3278 bytes --]
My time on QEMU is rather limited and I could hardly look at testing
patches for long. So it makes sense to drop me.
Acked-by: Fam Zheng <fam@euphon.net>
Thanks!
Fam
On Mon, 8 Mar 2021 at 15:50, Thomas Huth <thuth@redhat.com> 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
> M: Alex Bennée <alex.bennee@linaro.org>
> -M: Fam Zheng <fam@euphon.net>
> -R: Philippe Mathieu-Daudé <philmd@redhat.com>
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> +M: Thomas Huth <thuth@redhat.com>
> +R: Wainer dos Santos Moschetta <wainersm@redhat.com>
> S: Maintained
> F: .github/lockdown.yml
> +F: .gitlab-ci.yml
> +F: .gitlab-ci.d/
> F: .travis.yml
> F: scripts/ci/
> F: tests/docker/
> F: tests/vm/
> F: scripts/archive-source.sh
> +W: https://gitlab.com/qemu-project/qemu/pipelines
> W: https://travis-ci.org/qemu/qemu
> W: http://patchew.org/QEMU/
>
> @@ -3289,17 +3293,6 @@ S: Maintained
> F: .cirrus.yml
> W: https://cirrus-ci.com/github/qemu/qemu
>
> -GitLab Continuous Integration
> -M: Thomas Huth <thuth@redhat.com>
> -M: Philippe Mathieu-Daudé <philmd@redhat.com>
> -M: Alex Bennée <alex.bennee@linaro.org>
> -R: Wainer dos Santos Moschetta <wainersm@redhat.com>
> -S: Maintained
> -F: .gitlab-ci.yml
> -F: .gitlab-ci.d/crossbuilds.yml
> -F: .gitlab-ci.d/*py
> -F: scripts/ci/gitlab-pipeline-status
> -
> Guest Test Compilation Support
> M: Alex Bennée <alex.bennee@linaro.org>
> R: Philippe Mathieu-Daudé <f4bug@amsat.org>
> --
> 2.27.0
>
>
>
[-- Attachment #2: Type: text/html, Size: 5143 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-03-10 17:14 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-03-08 15:46 [PATCH v2] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section Thomas Huth
2021-03-08 16:35 ` Philippe Mathieu-Daudé
2021-03-08 17:01 ` Thomas Huth
2021-03-08 17:52 ` Philippe Mathieu-Daudé
2021-03-08 17:29 ` Wainer dos Santos Moschetta
2021-03-08 17:49 ` Willian Rampazzo
2021-03-08 19:12 ` Philippe Mathieu-Daudé
2021-03-09 12:45 ` Alex Bennée
2021-03-10 17:01 ` Fam Zheng
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.