[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-block] [PATCH 2/2] MAINTAINERS: Remove myself as
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [Qemu-block] [PATCH 2/2] MAINTAINERS: Remove myself as block maintainer |
Date: |
Wed, 09 Jan 2019 13:46:03 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Niels de Vos <address@hidden> writes:
> On Tue, Jan 08, 2019 at 08:12:16PM +0100, Kevin Wolf wrote:
>> Am 08.01.2019 um 18:18 hat Markus Armbruster geschrieben:
>> > 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.
>>
>> Orphan is probably by far closer to the truth than Supported. But I
>> think what we really do is Odd Fixes. Not sure if that's a status that
>> works in any meaningful way without having a maintainer for the specific
>> thing assigned?
>
> For Gluster we are interested in keeping it maintained in some form. At
> least address@hidden could be added as a list. I am not sure
> there is anyone working on Gluster that would fit for a maintainer of
> the driver. If you really want to put a name there, I can function as a
> 'maintainer' and pull in other Gluster developers in case I am otherwise
> occupied.
Leads us to the question what it takes to be a maintainer. I figure we
need at least a commitment to get patches reviewed, plus the ability to
get them reviewed competently.
Note my careful use of the passive voice: whether the maintainer does it
himself or herds a team of intelligent cats to do it may not matter all
that much to us.