-
Notifications
You must be signed in to change notification settings - Fork 2.8k
[exporter/loki] Bring lokiexporter back to contrib distribuition #38993
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/loki] Bring lokiexporter back to contrib distribuition #38993
Conversation
8b79b38
to
6bbace2
Compare
@@ -6,13 +6,14 @@ | |||
| Stability | [deprecated]: logs | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to make it as not deprecated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe it still should be deprecated, we intend to have as many users migrate to otlp/otlphttp exporters as possible, however there are still some issues with loki's otlp ingestion that prevent some users from making that migration yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, approved but this should have a changelog.
…n-telemetry#38993) <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description This is the follow-up PR to open-telemetry#38438. We at Grafana Labs still want the loki exporter in the contrib distribution. It was removed from the distribution accidentally, it should stay <!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. --> #### Link to tracking issue Fixes open-telemetry#38374
Description
This is the follow-up PR to #38438.
We at Grafana Labs still want the loki exporter in the contrib distribution. It was removed from the distribution accidentally, it should stay
Link to tracking issue
Fixes #38374