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

feat(parsers.openmetrics): Add parser for OpenMetrics format #15298

Merged
merged 17 commits into from
May 16, 2024

Conversation

srebhan
Copy link
Member

@srebhan srebhan commented May 6, 2024

Summary

This PR adds an OpenMetrics parser and integrates it into the prometheus input plugin. To allow more control over the processing of incoming messages, the PR also adds a content_type_override option to define the kind of messages expected for cases where the endpoint does not send a content-type or sends a wrong one (e.g. prometheus-text instead of openmetrics-text).

Checklist

  • No AI generated code was used in this PR

Related issues

resolves #9547

@srebhan srebhan changed the title Openmetrics issue 9547 feat(parsers.openmetrics): Add parser for OpenMetrics format May 6, 2024
@telegraf-tiger telegraf-tiger bot added feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin plugin/parser 1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins labels May 6, 2024
@telegraf-tiger
Copy link
Contributor

telegraf-tiger bot commented May 6, 2024

@srebhan srebhan self-assigned this May 6, 2024
@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label May 15, 2024
@srebhan srebhan assigned powersj and DStrand1 and unassigned srebhan May 15, 2024
Copy link
Contributor

@powersj powersj left a comment

Choose a reason for hiding this comment

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

Thanks for driving this!

@powersj powersj removed their assignment May 15, 2024
@DStrand1 DStrand1 merged commit 4896384 into influxdata:master May 16, 2024
27 checks passed
@github-actions github-actions bot added this to the v1.31.0 milestone May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin plugin/parser 1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for OpenMetrics standard
3 participants