Move rmfitzpatrick to emeritus (#35772)

Thanks @rmfitzpatrick for your contributions to the collector! After discussing a bit with Ryan, it appears the best route is to move him to emeritus and explicitly make way for new codeowners to take over and help. @rmfitzpatrick please review and approve. --------- Co-authored-by: Dmitrii Anoshin <[email protected]> Co-authored-by: Sean Marciniak <[email protected]>
open-telemetry · Oct 15, 2024 · 53b5dd7 · 53b5dd7
1 parent 2295a09
commit 53b5dd7
Show file tree

Hide file tree

Showing 20 changed files with 49 additions and 30 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -100,16 +100,16 @@ extension/googleclientauthextension/                                @open-teleme
 extension/headerssetterextension/                                   @open-telemetry/collector-contrib-approvers @jpkrohling
 extension/healthcheckextension/                                     @open-telemetry/collector-contrib-approvers @jpkrohling
 extension/healthcheckv2extension/                                   @open-telemetry/collector-contrib-approvers @jpkrohling @mwear
-extension/httpforwarderextension/                                   @open-telemetry/collector-contrib-approvers @atoulme @rmfitzpatrick
+extension/httpforwarderextension/                                   @open-telemetry/collector-contrib-approvers @atoulme
 extension/jaegerremotesampling/                                     @open-telemetry/collector-contrib-approvers @yurishkuro @frzifus
 extension/oauth2clientauthextension/                                @open-telemetry/collector-contrib-approvers @pavankrish123 @jpkrohling
-extension/observer/                                                 @open-telemetry/collector-contrib-approvers @dmitryax @rmfitzpatrick
+extension/observer/                                                 @open-telemetry/collector-contrib-approvers @dmitryax
 extension/observer/cfgardenobserver/                                @open-telemetry/collector-contrib-approvers @crobert-1 @cemdk @tomasmota @m1rp @jriguera
 extension/observer/dockerobserver/                                  @open-telemetry/collector-contrib-approvers @MovieStoreGuy
-extension/observer/ecsobserver/                                     @open-telemetry/collector-contrib-approvers @dmitryax @rmfitzpatrick
-extension/observer/ecstaskobserver/                                 @open-telemetry/collector-contrib-approvers @rmfitzpatrick
+extension/observer/ecsobserver/                                     @open-telemetry/collector-contrib-approvers @dmitryax
+extension/observer/ecstaskobserver/                                 @open-telemetry/collector-contrib-approvers
 extension/observer/hostobserver/                                    @open-telemetry/collector-contrib-approvers @MovieStoreGuy
-extension/observer/k8sobserver/                                     @open-telemetry/collector-contrib-approvers @rmfitzpatrick @dmitryax @ChrsMark
+extension/observer/k8sobserver/                                     @open-telemetry/collector-contrib-approvers @dmitryax @ChrsMark
 extension/oidcauthextension/                                        @open-telemetry/collector-contrib-approvers @jpkrohling
 extension/opampcustommessages/                                      @open-telemetry/collector-contrib-approvers @BinaryFissionGames @evan-bradley
 extension/opampextension/                                           @open-telemetry/collector-contrib-approvers @portertech @evan-bradley @tigrannajaryan @BinaryFissionGames
@@ -126,7 +126,7 @@ extension/sumologicextension/                                       @open-teleme
 internal/aws/                                                       @open-telemetry/collector-contrib-approvers @Aneurysm9 @mxiamxia
 internal/collectd/                                                  @open-telemetry/collector-contrib-approvers @atoulme
 internal/coreinternal/                                              @open-telemetry/collector-contrib-approvers @open-telemetry/collector-approvers
-internal/docker/                                                    @open-telemetry/collector-contrib-approvers @rmfitzpatrick @jamesmoessis
+internal/docker/                                                    @open-telemetry/collector-contrib-approvers @jamesmoessis
 internal/exp/metrics/                                               @open-telemetry/collector-contrib-approvers @sh0rez @RichieSams
 internal/filter/                                                    @open-telemetry/collector-contrib-approvers @open-telemetry/collector-approvers
 internal/grpcutil/                                                  @open-telemetry/collector-contrib-approvers @jmacd @moh-osman3 @lquerel
@@ -146,7 +146,7 @@ internal/tools/                                                     @open-teleme
 pkg/batchperresourceattr/                                           @open-telemetry/collector-contrib-approvers @atoulme @dmitryax
 pkg/batchpersignal/                                                 @open-telemetry/collector-contrib-approvers @jpkrohling
 pkg/datadog/                                                        @open-telemetry/collector-contrib-approvers @mx-psi @dineshg13 @liustanley @songy23 @mackjmr @ankitpatel96
-pkg/experimentalmetricmetadata/                                     @open-telemetry/collector-contrib-approvers @rmfitzpatrick
+pkg/experimentalmetricmetadata/                                     @open-telemetry/collector-contrib-approvers @dmitryax
 pkg/golden/                                                         @open-telemetry/collector-contrib-approvers @djaglowski @atoulme
 pkg/kafka/topic/                                                    @open-telemetry/collector-contrib-approvers @pavolloffay @MovieStoreGuy
 pkg/ottl/                                                           @open-telemetry/collector-contrib-approvers @TylerHelmuth @kentquirk @bogdandrutu @evan-bradley
@@ -179,7 +179,7 @@ processor/geoipprocessor/                                           @open-teleme
 processor/groupbyattrsprocessor/                                    @open-telemetry/collector-contrib-approvers @rnishtala-sumo
 processor/groupbytraceprocessor/                                    @open-telemetry/collector-contrib-approvers @jpkrohling
 processor/intervalprocessor/                                        @open-telemetry/collector-contrib-approvers @RichieSams @sh0rez @djaglowski
-processor/k8sattributesprocessor/                                   @open-telemetry/collector-contrib-approvers @dmitryax @rmfitzpatrick @fatsheep9146 @TylerHelmuth
+processor/k8sattributesprocessor/                                   @open-telemetry/collector-contrib-approvers @dmitryax @fatsheep9146 @TylerHelmuth
 processor/logdedupprocessor/                                        @open-telemetry/collector-contrib-approvers @BinaryFissionGames @MikeGoldsmith @djaglowski
 processor/logstransformprocessor/                                   @open-telemetry/collector-contrib-approvers @djaglowski @dehaansa
 processor/metricsgenerationprocessor/                               @open-telemetry/collector-contrib-approvers @Aneurysm9
@@ -232,7 +232,7 @@ receiver/cloudfoundryreceiver/                                      @open-teleme
 receiver/collectdreceiver/                                          @open-telemetry/collector-contrib-approvers @atoulme
 receiver/couchdbreceiver/                                           @open-telemetry/collector-contrib-approvers @djaglowski
 receiver/datadogreceiver/                                           @open-telemetry/collector-contrib-approvers @boostchicken @gouthamve @jpkrohling @MovieStoreGuy
-receiver/dockerstatsreceiver/                                       @open-telemetry/collector-contrib-approvers @rmfitzpatrick @jamesmoessis
+receiver/dockerstatsreceiver/                                       @open-telemetry/collector-contrib-approvers @jamesmoessis
 receiver/elasticsearchreceiver/                                     @open-telemetry/collector-contrib-approvers @djaglowski @BinaryFissionGames
 receiver/expvarreceiver/                                            @open-telemetry/collector-contrib-approvers @jamesmoessis @MovieStoreGuy
 receiver/filelogreceiver/                                           @open-telemetry/collector-contrib-approvers @djaglowski
@@ -248,7 +248,7 @@ receiver/httpcheckreceiver/                                         @open-teleme
 receiver/iisreceiver/                                               @open-telemetry/collector-contrib-approvers @Mrod1598 @pjanotti
 receiver/influxdbreceiver/                                          @open-telemetry/collector-contrib-approvers @jacobmarble
 receiver/jaegerreceiver/                                            @open-telemetry/collector-contrib-approvers @yurishkuro
-receiver/jmxreceiver/                                               @open-telemetry/collector-contrib-approvers @rmfitzpatrick
+receiver/jmxreceiver/                                               @open-telemetry/collector-contrib-approvers
 receiver/journaldreceiver/                                          @open-telemetry/collector-contrib-approvers @sumo-drosiek @djaglowski
 receiver/k8sclusterreceiver/                                        @open-telemetry/collector-contrib-approvers @dmitryax @TylerHelmuth @povilasv
 receiver/k8seventsreceiver/                                         @open-telemetry/collector-contrib-approvers @dmitryax @TylerHelmuth
@@ -277,7 +277,7 @@ receiver/pulsarreceiver/                                            @open-teleme
 receiver/purefareceiver/                                            @open-telemetry/collector-contrib-approvers @jpkrohling @dgoscn @chrroberts-pure
 receiver/purefbreceiver/                                            @open-telemetry/collector-contrib-approvers @jpkrohling @dgoscn @chrroberts-pure
 receiver/rabbitmqreceiver/                                          @open-telemetry/collector-contrib-approvers @djaglowski @cpheps
-receiver/receivercreator/                                           @open-telemetry/collector-contrib-approvers @rmfitzpatrick
+receiver/receivercreator/                                           @open-telemetry/collector-contrib-approvers @dmitryax
 receiver/redisreceiver/                                             @open-telemetry/collector-contrib-approvers @dmitryax @hughesjj
 receiver/riakreceiver/                                              @open-telemetry/collector-contrib-approvers @djaglowski @armstrmi
 receiver/saphanareceiver/                                           @open-telemetry/collector-contrib-approvers @dehaansa

diff --git a/extension/httpforwarderextension/README.md b/extension/httpforwarderextension/README.md
@@ -5,7 +5,8 @@
 | Stability     | [beta]  |
 | Distributions | [] |
 | Issues        | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aextension%2Fhttpforwarder%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Aextension%2Fhttpforwarder) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aextension%2Fhttpforwarder%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Aextension%2Fhttpforwarder) |
-| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@atoulme](https://www.github.com/atoulme), [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@atoulme](https://www.github.com/atoulme) |
+| Emeritus      | [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
 
 [beta]: https://github.com/open-telemetry/opentelemetry-collector#beta
 <!-- end autogenerated section -->

diff --git a/extension/httpforwarderextension/metadata.yaml b/extension/httpforwarderextension/metadata.yaml
@@ -6,7 +6,8 @@ status:
     beta: [extension]
   distributions: []
   codeowners:
-    active: [atoulme, rmfitzpatrick]
+    active: [atoulme]
+    emeritus: [rmfitzpatrick]
 
 tests:
   config:

diff --git a/extension/observer/ecsobserver/README.md b/extension/observer/ecsobserver/README.md
@@ -6,7 +6,8 @@
 | Stability     | [beta]  |
 | Distributions | [contrib] |
 | Issues        | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aextension%2Fecsobserver%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Aextension%2Fecsobserver) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aextension%2Fecsobserver%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Aextension%2Fecsobserver) |
-| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@dmitryax](https://www.github.com/dmitryax), [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@dmitryax](https://www.github.com/dmitryax) |
+| Emeritus      | [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
 
 [beta]: https://github.com/open-telemetry/opentelemetry-collector#beta
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/extension/observer/ecsobserver/metadata.yaml b/extension/observer/ecsobserver/metadata.yaml
@@ -6,7 +6,8 @@ status:
     beta: [extension]
   distributions: [contrib]
   codeowners:
-    active: [dmitryax, rmfitzpatrick]
+    active: [dmitryax]
+    emeritus: [rmfitzpatrick]
 
 # TODO: Update the extension to make the tests pass
 tests:

diff --git a/extension/observer/ecstaskobserver/README.md b/extension/observer/ecstaskobserver/README.md
@@ -6,7 +6,8 @@
 | Stability     | [beta]  |
 | Distributions | [contrib] |
 | Issues        | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aextension%2Fecstaskobserver%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Aextension%2Fecstaskobserver) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aextension%2Fecstaskobserver%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Aextension%2Fecstaskobserver) |
-| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    |  |
+| Emeritus      | [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
 
 [beta]: https://github.com/open-telemetry/opentelemetry-collector#beta
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/extension/observer/ecstaskobserver/metadata.yaml b/extension/observer/ecstaskobserver/metadata.yaml
@@ -6,7 +6,8 @@ status:
     beta: [extension]
   distributions: [contrib]
   codeowners:
-    active: [rmfitzpatrick]
+    active: []
+    emeritus: [rmfitzpatrick]
 
 # TODO: Update the extension to make the tests pass
 tests:

diff --git a/extension/observer/k8sobserver/README.md b/extension/observer/k8sobserver/README.md
@@ -6,7 +6,8 @@
 | Stability     | [alpha]  |
 | Distributions | [contrib] |
 | Issues        | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aextension%2Fk8sobserver%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Aextension%2Fk8sobserver) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aextension%2Fk8sobserver%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Aextension%2Fk8sobserver) |
-| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@rmfitzpatrick](https://www.github.com/rmfitzpatrick), [@dmitryax](https://www.github.com/dmitryax), [@ChrsMark](https://www.github.com/ChrsMark) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@dmitryax](https://www.github.com/dmitryax), [@ChrsMark](https://www.github.com/ChrsMark) |
+| Emeritus      | [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
 
 [alpha]: https://github.com/open-telemetry/opentelemetry-collector#alpha
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/extension/observer/k8sobserver/metadata.yaml b/extension/observer/k8sobserver/metadata.yaml
@@ -6,7 +6,8 @@ status:
     alpha: [extension]
   distributions: [contrib]
   codeowners:
-    active: [rmfitzpatrick, dmitryax, ChrsMark]
+    active: [dmitryax, ChrsMark]
+    emeritus: [rmfitzpatrick]
 
 # TODO: Update the extension to make the tests pass
 tests:

diff --git a/extension/observer/metadata.yaml b/extension/observer/metadata.yaml
@@ -1,3 +1,4 @@
 status:
   codeowners:
-    active: [dmitryax, rmfitzpatrick]
+    active: [dmitryax]
+    emeritus: [rmfitzpatrick]
diff --git a/internal/docker/metadata.yaml b/internal/docker/metadata.yaml
@@ -1,3 +1,4 @@
 status:
   codeowners:
-    active: [rmfitzpatrick, jamesmoessis]
+    active: [jamesmoessis, moviestoreguy]
+    emeritus: [rmfitzpatrick]
diff --git a/pkg/experimentalmetricmetadata/metadata.yaml b/pkg/experimentalmetricmetadata/metadata.yaml
@@ -1,3 +1,4 @@
 status:
   codeowners:
-    active: [rmfitzpatrick]
+    active: [dmitryax]
+    emeritus: [rmfitzpatrick]
diff --git a/processor/k8sattributesprocessor/README.md b/processor/k8sattributesprocessor/README.md
@@ -5,7 +5,8 @@
 | Stability     | [beta]: logs, metrics, traces   |
 | Distributions | [contrib] |
 | Issues        | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aprocessor%2Fk8sattributes%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Aprocessor%2Fk8sattributes) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aprocessor%2Fk8sattributes%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Aprocessor%2Fk8sattributes) |
-| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@dmitryax](https://www.github.com/dmitryax), [@rmfitzpatrick](https://www.github.com/rmfitzpatrick), [@fatsheep9146](https://www.github.com/fatsheep9146), [@TylerHelmuth](https://www.github.com/TylerHelmuth) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner)    | [@dmitryax](https://www.github.com/dmitryax), [@fatsheep9146](https://www.github.com/fatsheep9146), [@TylerHelmuth](https://www.github.com/TylerHelmuth) |
+| Emeritus      | [@rmfitzpatrick](https://www.github.com/rmfitzpatrick) |
 
 [beta]: https://github.com/open-telemetry/opentelemetry-collector#beta
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/processor/k8sattributesprocessor/metadata.yaml b/processor/k8sattributesprocessor/metadata.yaml
@@ -6,7 +6,8 @@ status:
     beta: [logs, metrics, traces]
   distributions: [contrib]
   codeowners:
-    active: [dmitryax, rmfitzpatrick, fatsheep9146, TylerHelmuth]
+    active: [dmitryax, fatsheep9146, TylerHelmuth]
+    emeritus: [rmfitzpatrick]
 # resource attributes are exposed through a different configuration interface (extract::metadata).
 resource_attributes:
   k8s.cluster.uid: