Skip to content

Commit

Permalink
CODEOWNERS: assign code ownership to florianl, mmat11 and dylandreime…
Browse files Browse the repository at this point in the history
…rink

As discussed at the Cilium Dev Summit 2024, this codifies review assignments
and expectations around interactivity on PR reviews.

Signed-off-by: Timo Beckers <timo@isovalent.com>
  • Loading branch information
ti-mo committed Mar 21, 2024
1 parent 42cbe8f commit 9330be4
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
7 changes: 7 additions & 0 deletions CODEOWNERS
Original file line number Diff line number Diff line change
@@ -1,2 +1,9 @@
* @cilium/ebpf-lib-maintainers

features/ @rgo3
link/ @mmat11

perf/ @florianl
ringbuf/ @florianl

btf/ @dylandreimerink
9 changes: 6 additions & 3 deletions docs/ebpf/contributing/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ from upstream kernel versions. You can update them to the latest version by:
Finally, bump the tested kernels in `.github/workflows/ci.yml`
## Project permissions
## Project Roles
If you'd like to contribute to the library more regularly, one of the
[maintainers][ebpf-lib-maintainers] can add you to the appropriate team or mark
Expand All @@ -81,10 +81,13 @@ you as a code owner. Please create an issue in the repository.
* Have ["Write"][permissions] role
* CODEOWNER of a part of the code base
* In-depth review of code, escalates to maintainers if necessary
* For bugfixes: review within 1-2 days
* Otherwise: review within a work week
* When lacking time: escalate to maintainers, but don’t ignore
* [ebpf-lib-maintainers]
* Have ["Admin"][permissions] role
* Manage releases
* Triage incoming issues and discussions
* Triage incoming issues and discussions and pull in CODEOWNERS if needed
* Maintain CI & project permissions
* Maintain roadmap and encourage contributions towards it
* Merge approved PRs
Expand All @@ -94,4 +97,4 @@ you as a code owner. Please create an issue in the repository.
[permissions]: https://docs.github.com/en/organizations/managing-user-access-to-your-organizations-repositories/repository-roles-for-an-organization#permissions-for-each-role
[ebpf-go-contributors]: https://github.com/orgs/cilium/teams/ebpf-go-contributors/members
[ebpf-go-reviewers]: https://github.com/orgs/cilium/teams/ebpf-go-reviewers/members
[ebpf-lib-maintainers]: https://github.com/orgs/cilium/teams/ebpf-lib-maintainers/members
[ebpf-lib-maintainers]: https://github.com/orgs/cilium/teams/ebpf-lib-maintainers/members

0 comments on commit 9330be4

Please sign in to comment.