Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Docker Stats Receiver] Add @jamesmoessis as code owner #16253

Merged

Conversation

@MovieStoreGuy MovieStoreGuy requested review from a team and TylerHelmuth November 11, 2022 00:40
@dmitryax dmitryax added Skip Changelog PRs that do not require a CHANGELOG.md entry receiver/dockerstats labels Nov 11, 2022
Copy link
Member

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @rmfitzpatrick please leave your feedback as code owner

@dmitryax
Copy link
Member

@evan-bradley can you please take a look at the failing ping-owners job whenever you have a chance?

@jamesmoessis
Copy link
Contributor

Thanks @MovieStoreGuy for raising this. More than happy to be a codeowner and keep contributing and reviewing PRs for it 🙂

@evan-bradley
Copy link
Contributor

@dmitryax Please see #16254.

@MovieStoreGuy
Copy link
Contributor Author

I'll rebase once the change is in @evan-bradley

@bogdandrutu
Copy link
Member

@MovieStoreGuy please rebase, I think I merged the fix

@codeboten codeboten merged commit a9c93cf into open-telemetry:main Nov 14, 2022
mdelapenya added a commit to mdelapenya/opentelemetry-collector-contrib that referenced this pull request Nov 15, 2022
* main:
  [chore] dependabot updates Tue Nov 15 00:17:56 UTC 2022 (open-telemetry#16300)
  [receiver/rabbitmq] Fix flaky integration test (open-telemetry#16240)
  [Docker Stats Receiver] Add @jamesmoessis as code owner (open-telemetry#16253)
  [receiver/mongodbatlas] Alerts poll mode check hostname and port (open-telemetry#16286)
  [chore] update jaeger dep (open-telemetry#16287)
  [pkg/stanza] Fix issue where specifying a non-existent storage extension caused panic during shutdown. (open-telemetry#16213)
  [pkg/ottl] Rename ottllogs to ottllog (open-telemetry#16242)
  Register featuregate, otherwise does not work (open-telemetry#16269)
  [receiver/elasticsearch]: add fielddata memory size metrics on index level (open-telemetry#14922)
  [chore] remove what looks to be debugging code from instanaexporter (open-telemetry#16258)
JaredTan95 pushed a commit to openinsight-proj/opentelemetry-collector-contrib that referenced this pull request Nov 21, 2022
shalper2 pushed a commit to shalper2/opentelemetry-collector-contrib that referenced this pull request Dec 6, 2022
@plantfansam plantfansam mentioned this pull request Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
receiver/dockerstats Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants