-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[exporter/tanzuobservability] Update codeowners #23395
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
Hi @atoulme, our nearish-term plan is to deprecate and eventually delete the Tanzu Observerability Exporter. We have better ways to ingest native OTLP protobufs now, so OTel Collector users can just switch to the OTLP Exporter if needed. I believe @keep94 is the one who will be doing the deprecation PR, followed by the eventual deletion PR. |
Understood. Please open a new issue to deprecate the exporter and we can close this one, linking to it. |
I haven't heard back. so I will file the issue to deprecate and eventually delete the exporter from your comment. Thanks! |
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 Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been closed as inactive because it has been stale for 120 days with no activity. |
Component(s)
exporter/tanzuobservability
Describe the issue you're reporting
Please see #20868 for context.
The tanzuobservabilityexporter is lucky to count @keep94 and @thepeterstone as codeowners, 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.
The text was updated successfully, but these errors were encountered: