[igt-dev] [RFC i-g-t 0/1] Introduce MAINTAINERS file to IGT

[Petri Latvala]

On Mon, Jan 31, 2022 at 01:19:20PM -0500, Mark Yacoub wrote:
> On Mon, Jan 31, 2022 at 10:18 AM Sean Paul <seanpaul at chromium.org> wrote:
> >
> > On Mon, Jan 31, 2022 at 9:29 AM Rodrigo Siqueira
> > <Rodrigo.Siqueira at amd.com> wrote:
> > >
> > > Hi,
> > >
> > > IGT is becoming more generic, and multiple vendors are adopting it. For
> > > example, AMD fully upstream all of its internal changes last year, and
> > > now we are relying on IGT from the upstream. Additionally, thanks to the
> > > ChromeOS project, we have other vendors working on the KMS tests as
> > > well. At this point, I believe that we need some mechanism to better
> > > contact developers that care about some specific tests or even provide an
> > > easy way for other people to find the right stakeholder for dealing with
> > > some particular issue.
> > >
> >
> > Hi Siqueira,
> > Do you have a sense for how much of this is necessary once igt
> > transitions to gitlab merge requests (or what it looks like in that
> > new reality)? IIRC the reason igt is not using MR's is because of some
> > CI infrastructure only talking mailing list, I was under the
> > impression that was changing and MR's were the way forward.
> >
> > Sean
> >
> I still see value for a maitainer list, not just for code reviews.
> I find it as a useful tool for Point of Contact when a test fails.
> If we suddenly see a test failing on a certain device, we can look up
> who would knows best about it. Or even if there are questions about a
> specific thing in a test on a certain SoC.
> 
> @Petri, do we know if MRs are coming soon?

They are coming, but I don't have any timeframes to even
guesstimate. Martin and Arek were working on the patchwork-gitlab
bridge and I don't have recent updates on that effort.

-- 
Petri Latvala


> > > Imho Linux Kernel already has a good solution codified in the
> > > MAINTAINERS and get_maintainers files. One of the best things about this
> > > approach was adding reviewers and maintainers (and other options) to the
> > > specific part of the code and using the get_maintainers tool to extract
> > > the right people to be part of the series. With this idea in mind, I
> > > created this RFC that introduces the MAINTAINERS file in IGT, and I have
> > > already added some entries to this file. My approach was:
> > >
> > > * Petri and Hiler are the maintainers, and for this reason, their
> > >   contact will be displayed for every patch.
> > > * I add Mark Yacoub as a reviewer for every KMS test.
> > > * I added some AMD folks as reviewers in some specific KMS tests that we
> > >   run in our CI.
> > > * I added some AMD maintainers to amdgpu tests.
> > >
> > > If you want to try it in your local repository, you will need the
> > > get_maintainers script from the Linux kernel, and you can use this
> > > command:
> > >
> > >  perl /PATH/TO/get_maintainer.pl --no-tree --separator , --nokeywords --nogit --nogit-fallback --norolestats -f <TARGET_TEST>
> > >
> > > Anyway, this is just an idea that I want to discuss with the community.
> > > What do you think about having a MAINTAINERs file? Do you want to add
> > > your name as a reviewer to some specific test if you like the idea? Any
> > > other comments or ideas?
> > >
> > > Cc: Arkadiusz Hiler <arek at hiler.eu>
> > > Cc: Petri Latvala <petri.latvala at intel.com>
> > > Cc: Mark Yacoub <markyacoub at google.com>
> > > Cc: Jessica Zhang <quic_jesszhan at quicinc.com>
> > > Cc: Abhinav Kumar <quic_abhinavk at quicinc.com>
> > > Cc: Melissa Wen <mwen at igalia.com>
> > > Cc: Sean Paul <seanpaul at chromium.org>
> > > Cc: Harry Wentland <harry.wentland at amd.com>
> > > Cc: Sun Peng Li(Leo) <sunpeng.li at amd.com>
> > > Cc: Chao-kai Wang (Stylon) <stylon.wang at amd.com>
> > > Cc: Wayne Lin <wayne.lin at amd.com>
> > > Cc: Nicholas Choi <nicholas.choi at amd.com>
> > > Cc: Martin Peres <martin.peres at mupuf.org>
> > > Cc: Aurabindo Pillai <aurabindo.pillai at amd.com>
> > > Cc: Bhawanpreet Lakha <bhawanpreet.lakha at amd.com>
> > > Cc: Qingqing Zhuo (Lilian) <qingqing.zhuo at amd.com>
> > > Cc: Solomon Chiu <solomon.chiu at amd.com>
> > >
> > > Best Regards,
> > > Siqueira
> > >
> > > Rodrigo Siqueira (1):
> > >   MAINTAINERS: Introduce MAINTAINERS file
> > >
> > >  MAINTAINERS | 203 +++++++++++++++++++++++++++++++++++++++++++++++++++-
> > >  1 file changed, 201 insertions(+), 2 deletions(-)
> > >
> > > --
> > > 2.25.1
> > >