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

[chore] add deprecation info for loki exporter #38311

Merged
merged 2 commits into from
Mar 3, 2025

Conversation

ChrsMark
Copy link
Member

@ChrsMark ChrsMark commented Mar 3, 2025

Description
See open-telemetry/opentelemetry-collector#12359

Add more deprecation metadata to the loki exporter.

/cc @atoulme

Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
@ChrsMark ChrsMark requested a review from a team as a code owner March 3, 2025 17:33
@ChrsMark ChrsMark requested a review from edmocosta March 3, 2025 17:33
@github-actions github-actions bot added the exporter/loki Loki Exporter label Mar 3, 2025
@github-actions github-actions bot requested review from gramidt and mar4uk March 3, 2025 17:33
@ChrsMark ChrsMark requested a review from atoulme March 3, 2025 17:33
Copy link
Contributor

@atoulme atoulme left a comment

Choose a reason for hiding this comment

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

Changes LGTM

@atoulme
Copy link
Contributor

atoulme commented Mar 3, 2025

Looks like CI is broken elsewhere.

@atoulme
Copy link
Contributor

atoulme commented Mar 3, 2025

Changes look good, just need codeowners to review and approve nevermind, this is not blocked on their review, merging.

@atoulme atoulme merged commit 5346f7b into open-telemetry:main Mar 3, 2025
156 checks passed
@github-actions github-actions bot added this to the next release milestone Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exporter/loki Loki Exporter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants