mbox series

[net,v2,0/7] MAINTAINERS: remove inactive folks from networking

Message ID 20210114014912.2519931-1-kuba@kernel.org (mailing list archive)
Headers show
Series MAINTAINERS: remove inactive folks from networking | expand

Message

Jakub Kicinski Jan. 14, 2021, 1:49 a.m. UTC
Hi!

This series intends to remove some most evidently inactive maintainers.

To make maintainers' lives easier we're trying to nudge people
towards CCing all the relevant folks on patches, in an attempt
to improve review rate. We have a check in patchwork which validates
the CC list against get_maintainers.pl. It's a little awkward, however,
to force people to CC maintainers who we haven't seen on the mailing
list for years. This series removes from maintainers folks who didn't
provide any tag (incl. authoring a patch) in the last 5 years.
To ensure reasonable signal to noise ratio we only considered
MAINTAINERS entries which had more than 100 patches fall under
them in that time period.

All this is purely a process-greasing exercise, I hope nobody
sees this series as an affront. Most folks are moved to CREDITS,
a couple entries are simply removed. 

The following inactive maintainers are kept, because they indicated
the intention to come back in the near future:

 - Veaceslav Falico (bonding)
 - Christian Benvenuti (Cisco drivers)
 - Felix Fietkau (mtk-eth)
 - Mirko Linder (skge/sky2)

Patches in this series contain report from a script which did
the analysis. Big thanks to Jonathan Corbet for help and writing
the script (although I feel like I used it differently than Jon
may have intended ;)). The output format is thus:

 Subsystem $name
  Changes $reviewed / $total ($percent%)           // how many changes to the subsystem had at least one ack/review
  Last activity: $date_of_most_recent_patch
  $maintainer/reviewer1:
    Author $last_commit_authored_by_the_person $how_many_in_5yrs
    Committer $last_committed $how_many
    Tags $last_tag_like_review_signoff_etc $how_many
  $maintainer/reviewer2:
    Author $last_commit_authored_by_the_person $how_many_in_5yrs
    Committer $last_committed $how_many
    Tags $last_tag_like_review_signoff_etc $how_many
  Top reviewers: // Top 3 reviewers (who are not listed in MAINTAINERS)
    [$count_of_reviews_and_acks]: $email
  INACTIVE MAINTAINER $name   // maintainer / reviewer who has done nothing in last 5yrs

v2:
 - keep Felix and Mirko

Jakub Kicinski (7):
  MAINTAINERS: altx: move Jay Cliburn to CREDITS
  MAINTAINERS: net: move Alexey Kuznetsov to CREDITS
  MAINTAINERS: vrf: move Shrijeet to CREDITS
  MAINTAINERS: ena: remove Zorik Machulsky from reviewers
  MAINTAINERS: tls: move Aviad to CREDITS
  MAINTAINERS: ipvs: move Wensong Zhang to CREDITS
  MAINTAINERS: dccp: move Gerrit Renker to CREDITS

 CREDITS     | 24 ++++++++++++++++++++++++
 MAINTAINERS |  9 +--------
 2 files changed, 25 insertions(+), 8 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Jan. 14, 2021, 7 p.m. UTC | #1
Hello:

This series was applied to netdev/net.git (refs/heads/master):

On Wed, 13 Jan 2021 17:49:05 -0800 you wrote:
> Hi!
> 
> This series intends to remove some most evidently inactive maintainers.
> 
> To make maintainers' lives easier we're trying to nudge people
> towards CCing all the relevant folks on patches, in an attempt
> to improve review rate. We have a check in patchwork which validates
> the CC list against get_maintainers.pl. It's a little awkward, however,
> to force people to CC maintainers who we haven't seen on the mailing
> list for years. This series removes from maintainers folks who didn't
> provide any tag (incl. authoring a patch) in the last 5 years.
> To ensure reasonable signal to noise ratio we only considered
> MAINTAINERS entries which had more than 100 patches fall under
> them in that time period.
> 
> [...]

Here is the summary with links:
  - [net,v2,1/7] MAINTAINERS: altx: move Jay Cliburn to CREDITS
    https://git.kernel.org/netdev/net/c/93089de91e85
  - [net,v2,2/7] MAINTAINERS: net: move Alexey Kuznetsov to CREDITS
    https://git.kernel.org/netdev/net/c/09cd3f4683a9
  - [net,v2,3/7] MAINTAINERS: vrf: move Shrijeet to CREDITS
    https://git.kernel.org/netdev/net/c/5e62d124f75a
  - [net,v2,4/7] MAINTAINERS: ena: remove Zorik Machulsky from reviewers
    https://git.kernel.org/netdev/net/c/c41efbf2ad56
  - [net,v2,5/7] MAINTAINERS: tls: move Aviad to CREDITS
    https://git.kernel.org/netdev/net/c/0e4ed0b62b5a
  - [net,v2,6/7] MAINTAINERS: ipvs: move Wensong Zhang to CREDITS
    https://git.kernel.org/netdev/net/c/4f3786e01194
  - [net,v2,7/7] MAINTAINERS: dccp: move Gerrit Renker to CREDITS
    https://git.kernel.org/netdev/net/c/054c4610bd05

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html