[LU-11329] add get_maintainer.pl script from kernel Created: 03/Sep/18 Updated: 30/Jan/22 Resolved: 27/Nov/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.12.0, Lustre 2.13.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Andreas Dilger | Assignee: | Andreas Dilger |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| 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. |
| Comments |
| Comment by Gerrit Updater [ 03/Sep/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33104 |
| Comment by Gerrit Updater [ 21/Sep/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33104/ |
| Comment by Gerrit Updater [ 12/Oct/18 ] |
|
James Nunez (jnunez@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33360 |
| Comment by Gerrit Updater [ 16/Oct/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) merged in patch https://review.whamcloud.com/33360/ |
| Comment by Gerrit Updater [ 22/Oct/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33413 |
| Comment by Gerrit Updater [ 24/Oct/18 ] |
|
Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/33467 |
| Comment by Gerrit Updater [ 02/Nov/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33413/ |
| Comment by Gerrit Updater [ 02/Nov/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33467/ |
| Comment by Gerrit Updater [ 19/Nov/18 ] |
|
Patrick Farrell (paf@cray.com) uploaded a new patch: https://review.whamcloud.com/33684 |
| Comment by Peter Jones [ 25/Nov/18 ] |
|
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 |
| Comment by Andreas Dilger [ 25/Nov/18 ] |
|
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. |
| Comment by Gerrit Updater [ 27/Nov/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33684/ |
| Comment by Peter Jones [ 27/Nov/18 ] |
|
ok fair enough - landed for 2.12 |
| Comment by Gerrit Updater [ 25/Jan/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/34109 |
| Comment by Gerrit Updater [ 08/Mar/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34109/ |