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] update loki exporter to use telemetry builder #33098

Merged

Conversation

codeboten
Copy link
Contributor

This uses the new mdatagen functionality to define all internal telemetry for a component in metadata.yaml for ease of discovery, consistency across components, and in the near future simplify testing of telemetry produced by components.

@codeboten codeboten requested a review from jpkrohling as a code owner May 16, 2024 18:30
@codeboten codeboten requested a review from a team May 16, 2024 18:30
@github-actions github-actions bot added the exporter/loki Loki Exporter label May 16, 2024
@github-actions github-actions bot requested review from gouthamve, gramidt and mar4uk May 16, 2024 18:30
codeboten added 2 commits May 17, 2024 11:30
This uses the new mdatagen functionality to define all internal telemetry for a component
in metadata.yaml for ease of discovery, consistency across components, and in the near future
simplify testing of telemetry produced by components.

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@codeboten codeboten force-pushed the codeboten/loki-use-mdatagen-metrics branch from ebf44d8 to 57953ec Compare May 17, 2024 18:34
@codeboten codeboten merged commit f727003 into open-telemetry:main May 17, 2024
162 checks passed
@github-actions github-actions bot added this to the next release milestone May 17, 2024
@codeboten codeboten deleted the codeboten/loki-use-mdatagen-metrics branch May 21, 2024 15:12
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.

4 participants