[LU-11329] add get_maintainer.pl script from kernel

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.12.0, Lustre 2.13.0
• Assignee: Andreas Dilger
• Reporter: Andreas Dilger

Description

The get_maintainer.pl script can be given a patch file or list of pathnames and will look at the MAINTAINERS file for different code subsections and the developers that are maintaining those sections. If no maintainer is found, it will also check the git commit history and "git blame" to see which developers were active on the specified file(s) and use them.

It would be useful to have this in the Lustre tree to make it easier to find reviewers for the code.

Issue Links

• is duplicated by: LU-8075 Automatically add peer reviewers per the lustre.org community reviewers list
• is related to: LU-5879 Tell interested people when a section of code they are interested in has a proposed change

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/34109/
Subject: LU-11329 misc: reorder Git .mailmap file
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 0599ede3dab99eced0820541a806c7a23e3ec48b

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34109
Subject: LU-11329 misc: reorder Git .mailmap file
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 81af44c2b925137b05e963074018005c823efe6e

[Peter Jones]

ok fair enough - landed for 2.12

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33684/
Subject: LU-11329 utils: Add maintainer entries
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 82cccc51ffd282efd9835f01db8be0770a7e3c62

[Andreas Dilger]

I would say close this ticket when the last patch lands, and we can use a new ticket for updating the MAINTAINERS file. The addition of get_maintaibers.pl is done.

[Peter Jones]

I've removed the fix version because, even though there's no doubt that Patrick's patch will make it into 2.12, this seems like an open-ended ticket where there could be future patches from other sources

[Gerrit Updater]

Patrick Farrell ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33684
Subject: LU-11329 utils: Add maintainer entries
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e48654a5fe79f99d7f0ef5f55ffbfe57b64c3688

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33467/
Subject: LU-11329 misc: Add LLNL reviewers to MAINTAINERS
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 86bb350708ed035309466286152707f3a1d3a1a1

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33413/
Subject: LU-11329 misc: populate MAINTAINERS file
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 92b664139c1d075841a318cd4c23eb0169689a38

[Gerrit Updater]

Olaf Faaland-LLNL ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33467
Subject: LU-11329 misc: Add LLNL reviewers to MAINTAINERS
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d8959d64ab59b9e3db7269f974c979e23404947d