From: Florian Forster <octo@collectd.org>
Date: Mon, 17 Feb 2020 14:31:18 +0000 (+0100)
Subject: CODEOWNERS: Add "core maintainers" owning daemon and important plugins.
X-Git-Tag: collectd-5.12.0~57
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=62ae49f47745709e9498bf9998887a87e22d6ea2;p=thirdparty%2Fcollectd.git

CODEOWNERS: Add "core maintainers" owning daemon and important plugins.
---

diff --git a/CODEOWNERS b/CODEOWNERS
index 5a2fe301e..f60ab20ba 100644
--- a/CODEOWNERS
+++ b/CODEOWNERS
@@ -1,18 +1,46 @@
 # Code ownership information.
-# See https://help.github.com/articles/about-code-owners/ for details.
+# See
+# https://help.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners
+# for details.
 
+# Order is important; the *last* matching pattern takes precedence.
+
+# Default
+#
 # These owners will be the default owners for everything in the repo. Unless a
-# later match takes precedence, # @trusted-contributors will be requested for
-# review when someone opens a pull request.
+# later match takes precedence, @collectd/trusted-contributors will be
+# requested for review when someone opens a pull request.
+*       @collectd/trusted-contributors
 
-/src/intel_pmu.c	@kwiatrox @sunkuranganath
-/src/intel_rdt.c	@kwiatrox @sunkuranganath
-/src/ipmi.c		@anaudx @rjablonx
-/src/mcelog.c		@kwiatrox @sunkuranganath
-/src/virt.c		@anaudx @rjablonx
+# Per-plugin owners
+#
+# These plugins are owned by subject matter experts and require their review.
+/src/intel_pmu.c	@collectd/intel
+/src/intel_rdt.c	@collectd/intel
+/src/ipmi.c		@collectd/intel
+/src/mcelog.c		@collectd/intel
+/src/virt.c		@collectd/intel
 # TODO(#2926): Add the following owners:
-#/src/redfish.c		@kkepka @mkobyli
+#/src/redfish.c		@collectd/intel
 
-# Order is important; the last matching pattern takes the most
-# precedence.
-*       @trusted-contributors
+# Core
+#
+# The daemon and some plugins with a huge "blast radius" are considered "core"
+# to the collectd project and require review form a "core owner".
+/CODEOWNERS		@collectd/core-maintainers
+/src/daemon/		@collectd/core-maintainers
+/src/liboconfig/	@collectd/core-maintainers
+/src/cpu.c		@collectd/core-maintainers
+/src/df.c		@collectd/core-maintainers
+/src/disk.c		@collectd/core-maintainers
+/src/exec.c		@collectd/core-maintainers
+/src/interface.c	@collectd/core-maintainers
+/src/memory.c		@collectd/core-maintainers
+/src/network.*		@collectd/core-maintainers
+/src/utils/avltree/	@collectd/core-maintainers
+/src/utils/common/	@collectd/core-maintainers
+/src/utils_fbhash.*	@collectd/core-maintainers
+/src/utils/heap/	@collectd/core-maintainers
+/src/utils/ignorelist/	@collectd/core-maintainers
+/src/utils/metadata/	@collectd/core-maintainers
+/src/utils/mount/	@collectd/core-maintainers