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.

Sign up for GitHub

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

Jump to bottom

[exporter/skywalking] Update codeowners #23397

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

[exporter/skywalking] Update codeowners #23397

atoulme opened this issue Jun 15, 2023 · 4 comments · Fixed by #30294
Labels
comp:skywalking exporter/skywalking Stale

Comments

@atoulme
Copy link
Contributor

atoulme commented Jun 15, 2023

Component(s)

exporter/skywalking

Describe the issue you're reporting

Please see #20868 for context.

The skywalkingexporter is lucky to count @liqiangz 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.
@atoulme atoulme added the needs triage New item requiring triage label Jun 15, 2023
@github-actions
Copy link
Contributor

Pinging code owners:

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

@atoulme atoulme mentioned this issue Jun 15, 2023
Closed
40 tasks
@codeboten codeboten removed the needs triage New item requiring triage label Jun 28, 2023
@github-actions
Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

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

@github-actions github-actions bot added the Stale label Sep 11, 2023
@atoulme atoulme removed the Stale label Oct 27, 2023
@atoulme
Copy link
Contributor Author

atoulme commented Oct 27, 2023

@liqiangz would you like to be removed from codeowners? Please let us know

@github-actions GitHub Actions
Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

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

@github-actions github-actions bot added the Stale label Dec 27, 2023
@atoulme atoulme mentioned this issue Jan 4, 2024
Merged
dmitryax pushed a commit that referenced this issue Jan 5, 2024
@atoulme
[chore][exporter/skywalking] remove codeowner (#30294)
669f23d 
**Description:**
Move @liqiangz  to emeritus codeowner.

**Link to tracking Issue:**
Fixes #23397

NOTE: This removes the last active codeowner of the skywalking exporter.
If this is merged, a follow up PR will be filed to move this component
to `unmaintained` stage, and eventually deprecation and removal.
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this issue Jan 10, 2024
@atoulme @cparkins
[chore][exporter/skywalking] remove codeowner (open-telemetry#30294)
d56172e 
**Description:**
Move @liqiangz  to emeritus codeowner.

**Link to tracking Issue:**
Fixes open-telemetry#23397

NOTE: This removes the last active codeowner of the skywalking exporter.
If this is merged, a follow up PR will be filed to move this component
to `unmaintained` stage, and eventually deprecation and removal.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
comp:skywalking exporter/skywalking Stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[chore][exporter/skywalking] remove codeowner atoulme/opentelemetry-collector-contrib
3 participants
@jpkrohling @atoulme @codeboten