feat: add maintainer drift detection feature#382
feat: add maintainer drift detection feature#382ahmadqasembader wants to merge 2 commits intocncf:mainfrom
Conversation
Signed-off-by: ahmadqasembader <ahmadbader245@gmail.com>
|
This is a more complicated feature. Maintainers are considered people who "speak for the project" and are designated through a project's governance model. Drift through PRs isn't a correct method to do this. Instead, one idea was to ping projects if a maintainer.yaml hasn't been updated in six months (via an issue) asking "is this still up to date? Also (if applicable) we see these $x people have been contributing heavily over the last six months and these $y people haven't touched anything in GitHub in the last six months" |
Signed-off-by: ahmadqasembader <ahmadbader245@gmail.com>
|
@jeefy, the drift detector has been refactored and introduced a The scheduled workflow will check the If yes, we will create an issue and tag the first 3 owners or the top 3 contributors of the Org. An example of the Issue that will be created: Maintainer List Health Check —
|
| Maintainer | Commits | Merged PRs | Active repos | Last seen |
|---|---|---|---|---|
@thockin |
312 | 71 | kubernetes, api, apimachinery +9 more | 2026-04-29 |
@liggitt |
278 | 64 | kubernetes, apiserver, client-go +4 more | 2026-04-28 |
@derekwaynecarr |
143 | 38 | kubernetes, component-base, api | 2026-04-22 |
@dims |
89 | 22 | kubernetes, apimachinery, api | 2026-03-15 |
@smarterclayton |
14 | 3 | apiserver | 2026-01-20 |
@lavalamp |
3 | 0 | kubernetes | 2025-11-04 |
@wojtek-t |
0 | 0 | — | — |
Active Contributors Not Yet in Maintainer List
These contributors have been active across project repos in the last 6 months but do not appear in maintainers.yaml. Consider whether any should be nominated per your governance process.
| Handle | Commits | Merged PRs | Active repos |
|---|---|---|---|
@bentheelder |
104 | 28 | kubernetes, client-go, api +1 more |
@pohly |
91 | 21 | apiserver, kubernetes |
@jpbetz |
77 | 18 | apiserver, apimachinery |
@alexzielenski |
55 | 14 | apiserver, api |
@gnufied |
43 | 9 | kubernetes, component-base |
No changes are required automatically. Please update
maintainers.yamlif this no longer reflects your governance, or close this issue to confirm it is still accurate. This issue was opened by cncf/automation maintainer health-check tooling.
I introduced a maintainer-list drift detection. It compares the org repos governance files against the
maintainers.ymlfile in.project, the detection works on every push or PR, in this case, if a drift is detected, a warning will be issued.The second part is a scheduled GitHub Actions that runs every 1st of the month, which tries to validate the maintainers again and open a PR if any drift was detected