Skip to content

Commit

Permalink
[chore] Remove @pmcollins as code owner (open-telemetry#31044)
Browse files Browse the repository at this point in the history
**Description:** <Describe what has changed.>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
This removes @pmcollins as code owner, as he reached out and is no
longer able to be a code owner in the collector repository.

Thanks for all of your help here Pablo!

**Link to tracking Issue:** <Issue number if applicable>
Should be merged after open-telemetry#31043.
  • Loading branch information
crobert-1 committed Feb 5, 2024
1 parent bf75f58 commit 0e7790b
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 8 deletions.
6 changes: 3 additions & 3 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

* @open-telemetry/collector-contrib-approvers

cmd/configschema/ @open-telemetry/collector-contrib-approvers @mx-psi @dmitryax @pmcollins
cmd/configschema/ @open-telemetry/collector-contrib-approvers @mx-psi @dmitryax
cmd/githubgen/ @open-telemetry/collector-contrib-approvers @atoulme
cmd/mdatagen/ @open-telemetry/collector-contrib-approvers @dmitryax
cmd/opampsupervisor/ @open-telemetry/collector-contrib-approvers @evan-bradley @atoulme @tigrannajaryan
Expand Down Expand Up @@ -203,7 +203,7 @@ receiver/dockerstatsreceiver/ @open-telemetry/collect
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
receiver/filereceiver/ @open-telemetry/collector-contrib-approvers @pmcollins @djaglowski
receiver/filereceiver/ @open-telemetry/collector-contrib-approvers @djaglowski
receiver/filestatsreceiver/ @open-telemetry/collector-contrib-approvers @atoulme
receiver/flinkmetricsreceiver/ @open-telemetry/collector-contrib-approvers @JonathanWamsley @djaglowski
receiver/fluentforwardreceiver/ @open-telemetry/collector-contrib-approvers @dmitryax
Expand Down Expand Up @@ -256,7 +256,7 @@ receiver/snowflakereceiver/ @open-telemetry/collect
receiver/solacereceiver/ @open-telemetry/collector-contrib-approvers @djaglowski @mcardy
receiver/splunkenterprisereceiver/ @open-telemetry/collector-contrib-approvers @shalper2 @MovieStoreGuy @greatestusername
receiver/splunkhecreceiver/ @open-telemetry/collector-contrib-approvers @atoulme
receiver/sqlqueryreceiver/ @open-telemetry/collector-contrib-approvers @dmitryax @pmcollins
receiver/sqlqueryreceiver/ @open-telemetry/collector-contrib-approvers @dmitryax
receiver/sqlserverreceiver/ @open-telemetry/collector-contrib-approvers @djaglowski @StefanKurek
receiver/sshcheckreceiver/ @open-telemetry/collector-contrib-approvers @nslaughter @codeboten
receiver/statsdreceiver/ @open-telemetry/collector-contrib-approvers @jmacd @dmitryax
Expand Down
3 changes: 2 additions & 1 deletion cmd/configschema/metadata.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,5 @@ type: configschema
status:
class: cmd
codeowners:
active: [mx-psi, dmitryax, pmcollins]
active: [mx-psi, dmitryax]
emeritus: [pmcollins]
3 changes: 2 additions & 1 deletion receiver/filereceiver/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@
| Stability | [development]: metrics |
| Distributions | [contrib] |
| Issues | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Areceiver%2Ffile%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Areceiver%2Ffile) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Areceiver%2Ffile%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Areceiver%2Ffile) |
| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@pmcollins](https://www.github.com/pmcollins), [@djaglowski](https://www.github.com/djaglowski) |
| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@djaglowski](https://www.github.com/djaglowski) |
| Emeritus | [@pmcollins](https://www.github.com/pmcollins) |

[development]: https://github.com/open-telemetry/opentelemetry-collector#development
[contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib
Expand Down
3 changes: 2 additions & 1 deletion receiver/filereceiver/metadata.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,5 @@ status:
development: [metrics]
distributions: [contrib]
codeowners:
active: [pmcollins, djaglowski]
active: [djaglowski]
emeritus: [pmcollins]
3 changes: 2 additions & 1 deletion receiver/sqlqueryreceiver/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@
| | [development]: logs |
| Distributions | [contrib], [observiq], [splunk], [sumo] |
| Issues | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Areceiver%2Fsqlquery%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aopen+is%3Aissue+label%3Areceiver%2Fsqlquery) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Areceiver%2Fsqlquery%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues?q=is%3Aclosed+is%3Aissue+label%3Areceiver%2Fsqlquery) |
| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@dmitryax](https://www.github.com/dmitryax), [@pmcollins](https://www.github.com/pmcollins) |
| [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 | [@pmcollins](https://www.github.com/pmcollins) |

[alpha]: https://github.com/open-telemetry/opentelemetry-collector#alpha
[development]: https://github.com/open-telemetry/opentelemetry-collector#development
Expand Down
3 changes: 2 additions & 1 deletion receiver/sqlqueryreceiver/metadata.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@ status:
development: [logs]
distributions: [contrib, splunk, observiq, sumo]
codeowners:
active: [dmitryax, pmcollins]
active: [dmitryax]
emeritus: [pmcollins]

tests:
config:

0 comments on commit 0e7790b

Please sign in to comment.