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

[receiver/pulsar] Update codeowners #23406

Closed
1 of 2 tasks
Tracked by #20868
atoulme opened this issue Jun 15, 2023 · 4 comments · Fixed by #24424
Closed
1 of 2 tasks
Tracked by #20868

[receiver/pulsar] Update codeowners #23406

atoulme opened this issue Jun 15, 2023 · 4 comments · Fixed by #24424

Comments

@atoulme
Copy link
Contributor

atoulme commented Jun 15, 2023

Component(s)

receiver/pulsar

Describe the issue you're reporting

Please see #20868 for context.

The pulsarreceiver is lucky to count @tjiuming as codeowner, however they are not currently registered as a member of OpenTelemetry.

Please work towards a resolution:

If no action is taken by January 2024, I will open a PR to remove them from the codeowners, and after a delay for their review, eventually will merge the PR.

@github-actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@codeboten codeboten removed the needs triage New item requiring triage label Jun 28, 2023
@tjiuming
Copy link
Contributor

@atoulme You can update the codeowner to @dao-jun , it's my another github account, and joined OTEL as a member.

@atoulme
Copy link
Contributor Author

atoulme commented Jul 12, 2023

I am sorry, I cannot do this on your behalf. Please open a PR to update the codeowners @tjiuming @dao-jun

@dao-jun
Copy link
Member

dao-jun commented Jul 13, 2023

OK,I'll do it

codeboten pushed a commit that referenced this issue Jul 21, 2023
Update pulsar receiver/exporter codeowner from @tjiuming to @dao-jun 

Fixes #23406
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants