Re: [PATCH 1/3] docs: add maintainer entry profile for XFS

[Luis Chamberlain <mcgrof@kernel.org>]

On Sat, Aug 12, 2023 at 12:05:33PM +0300, Amir Goldstein wrote:
> On Sat, Aug 12, 2023 at 3:04 AM Darrick J. Wong <djwong@kernel.org> wrote:
> >
> > On Fri, Aug 11, 2023 at 12:31:18PM -0700, Luis Chamberlain wrote:
> > > On Tue, Aug 01, 2023 at 12:58:21PM -0700, Darrick J. Wong wrote:
> > > > +Roles
> > > > +-----
> > > > +There are seven key roles in the XFS project.
> > > > +- **Testing Lead**: This person is responsible for setting the test
> > > > +  coverage goals of the project, negotiating with developers to decide
> > > > +  on new tests for new features, and making sure that developers and
> > > > +  release managers execute on the testing.
> > > > +
> > > > +  The testing lead should identify themselves with an ``M:`` entry in
> > > > +  the XFS section of the fstests MAINTAINERS file.
> > >
> > > I think breaking responsibility down is very sensible, and should hopefully
> > > allow you to not burn out. Given I realize how difficult it is to do all
> > > the tasks, and since I'm already doing quite a bit of testing of XFS
> > > on linux-next I can volunteer to help with this task of testing lead
> > > if folks also think it may be useful to the community.
> > >
> > > The only thing is I'd like to also ask if Amir would join me on the
> > > role to avoid conflicts of interest when and if it comes down to testing
> > > features I'm involved in somehow.
> >
> > Good question.  Amir?
> >
> 
> I am more than happy to help, but I don't believe that I currently perform
> or that I will have time to perform the official duties of **Testing
> Lead** role.
> 
> I fully support the nomination of Luis and I think the **Release Manager**
> should be able to resolve any conflict of interests of the **Testing Lead**
> as feature developer should any such conflicts arise :)

Fair enough.

Darrick, I suppose just one thing then, using M for Testing Lead seems
likely to implicate the 'Testing Lead' getting Cc'd on every single new
patch. As much as I could help review, I don't think I can commit to
that, and I think that's the point of the current split. To let us split
roles to help scale stuff.

So how about a separate new prefix, TL: ? Adding Linus in case he has
a stronger preference to only keep us at one character fist index on
MAINTAINERS.

  Luis