devapps/owners: add new owners for pprof & trace packages
pprof: cherryyz will be the primary owner
trace: mknyszek&prattmic will be the primary owner
The functionality depends on the runtime internal greatly
so it makes more sense that runtime team owns them.
This CL adds go/src/internal/{trace, profile} packages to
the list.
Currently rsc is the primary owner of
"go/src/net/http/pprof". I propose to add cherryyz to the
primary owner because this is the user-facing package
of pprof.
Change-Id: Ib5bcbfddf3b290943569c63e75e3ba57c40e3098
Reviewed-on: https://go-review.googlesource.com/c/build/+/297510
Trust: Hyang-Ah Hana Kim <hyangah@gmail.com>
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Jeremy Faller <jeremy@golang.org>
diff --git a/devapp/owners/table.go b/devapp/owners/table.go
index ae46261..c83b70c 100644
--- a/devapp/owners/table.go
+++ b/devapp/owners/table.go
@@ -205,10 +205,10 @@
Secondary: []Owner{mwhudson, thanm},
},
"go/src/cmd/pprof": {
- Primary: []Owner{hyangah},
+ Primary: []Owner{cherryyz},
},
"go/src/cmd/trace": {
- Primary: []Owner{hyangah},
+ Primary: []Owner{mknyszek, prattmic},
},
"go/src/cmd/vet": {
Primary: []Owner{adonovan},
@@ -369,6 +369,9 @@
"go/src/internal/cpu": {
Primary: []Owner{khr, martisch},
},
+ "go/src/internal/profile": {
+ Primary: []Owner{cherryyz},
+ },
"go/src/internal/singleflight": {
Primary: []Owner{bradfitz, iant},
},
@@ -392,6 +395,9 @@
Primary: []Owner{bradfitz, iant},
Secondary: []Owner{josharian},
},
+ "go/src/internal/trace": {
+ Primary: []Owner{mknyszek, prattmic},
+ },
"go/src/io": {
Primary: []Owner{gri},
Secondary: []Owner{iant, bradfitz},
@@ -431,7 +437,7 @@
Secondary: []Owner{rsc, empijei},
},
"go/src/net/http/pprof": {
- Primary: []Owner{rsc},
+ Primary: []Owner{cherryyz, rsc},
},
"go/src/net/internal/socktest": {
Primary: []Owner{},
@@ -503,17 +509,16 @@
Primary: []Owner{austin, khr},
},
"go/src/runtime/pprof": {
- Primary: []Owner{hyangah},
+ Primary: []Owner{cherryyz},
},
"go/src/runtime/pprof/internal/protopprof": {
- Primary: []Owner{},
- Secondary: []Owner{matloob},
+ Primary: []Owner{cherryyz},
},
"go/src/runtime/race": {
Primary: []Owner{dvyukov},
},
"go/src/runtime/trace": {
- Primary: []Owner{hyangah, dvyukov},
+ Primary: []Owner{mknyszek, prattmic, dvyukov},
},
"go/src/sort": {
Primary: []Owner{rsc, gri, iant, bradfitz},