[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH 2/2] MAINTAINERS: Remove myself as
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH 2/2] MAINTAINERS: Remove myself as block maintainer |
Date: |
Tue, 08 Jan 2019 18:18:18 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
This patch series got stuck.
Markus Armbruster <address@hidden> writes:
> Fam Zheng <address@hidden> writes:
>
>> On Tue, 09/25 07:00, Markus Armbruster wrote:
>>> Jeff Cody <address@hidden> writes:
>>>
>>> > I'll not be involved in day-to-day qemu development. Remove
>>> > myself as maintainer from the remainder of the network block drivers
>>> > (and vhdx), and revert them to the general block layer maintainership.
>>> >
>>> > Signed-off-by: Jeff Cody <address@hidden>
>>> > ---
>>> > MAINTAINERS | 14 --------------
>>> > 1 file changed, 14 deletions(-)
>>> >
>>> > diff --git a/MAINTAINERS b/MAINTAINERS
>>> > index e93f79672f..6ef6932628 100644
>>> > --- a/MAINTAINERS
>>> > +++ b/MAINTAINERS
>>> > @@ -1982,28 +1982,22 @@ F: block/vmdk.c
>>> >
>>> > RBD
>>> > M: Josh Durgin <address@hidden>
>>> > -M: Jeff Cody <address@hidden>
>>> > L: address@hidden
>>> > S: Supported
>>> > F: block/rbd.c
>>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block
>>> >
>>> > Sheepdog
>>> > M: Hitoshi Mitake <address@hidden>
>>> > M: Liu Yuan <address@hidden>
>>> > -M: Jeff Cody <address@hidden>
>>> > L: address@hidden
>>> > L: address@hidden
>>> > S: Supported
>>> > F: block/sheepdog.c
>>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block
>>> >
>>> > VHDX
>>> > -M: Jeff Cody <address@hidden>
>>> > L: address@hidden
>>> > S: Supported
>>> > F: block/vhdx*
>>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block
>>>
>>> Does "S: Supported" make sense without an M:?
>>>
>>> >
>>> > VDI
>>> > M: Stefan Weil <address@hidden>
>>> > @@ -2034,34 +2028,26 @@ F: docs/interop/nbd.txt
>>> > T: git git://repo.or.cz/qemu/ericb.git nbd
>>> >
>>> > NFS
>>> > -M: Jeff Cody <address@hidden>
>>> > M: Peter Lieven <address@hidden>
>>> > L: address@hidden
>>> > S: Maintained
>>> > F: block/nfs.c
>>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block
>>> >
>>> > SSH
>>> > M: Richard W.M. Jones <address@hidden>
>>> > -M: Jeff Cody <address@hidden>
>>> > L: address@hidden
>>> > S: Supported
>>> > F: block/ssh.c
>>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block
>>> >
>>> > CURL
>>> > -M: Jeff Cody <address@hidden>
>>> > L: address@hidden
>>> > S: Supported
>>> > F: block/curl.c
>>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block
>>>
>>> Likewise.
>>>
>>> > GLUSTER
>>> > -M: Jeff Cody <address@hidden>
>>> > L: address@hidden
>>> > S: Supported
>>> > F: block/gluster.c
>>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block
>>>
>>> Likewise.
>>>
>>> > Null Block Driver
>>> > M: Fam Zheng <address@hidden>
>>>
>>
>> Block drivers without an M: naturally fall under the overall maintainership
>> of
>> block layer (Kevin&Max), so IMO keeping the statuses is fine. Maybe CURL can
>> be
>> degraded to Maintained, though.
>
> Yes, get_maintainer.pl combines all applicable sections. Human readers
> are left to wonder, unless they know to look for other matches.
>
> Do we want to have a dedicated network block driver submaintainer again,
> if we can find one?
>
> Do we want to have a dedicated VHDX driver submaintainer again? Fam,
> you're maintaining VMDK, could you cover VHDX as well?
Fam can't.
I can see two sane options for the three sections that lose their sole
maintainer (VHDX, CURL, GLUSTER):
* Downgrade to S: Orphan
This reflects the fact that we'd like to have dedicated maintainers
for them. It camouflages the fact that the "Block layer core"
maintainers pick up the slack.
* Delete
The opposite.
Kevin, Max, please pick your poison, or suggest one you find tastier.
- Re: [Qemu-block] [Qemu-devel] [PATCH 2/2] MAINTAINERS: Remove myself as block maintainer,
Markus Armbruster <=