Skip to content

Fix prometheus metrics text exporter rendering#10

Merged
adam-rl merged 1 commit into
mainfrom
adam/metrics
Aug 18, 2025
Merged

Fix prometheus metrics text exporter rendering#10
adam-rl merged 1 commit into
mainfrom
adam/metrics

Conversation

@adam-rl

@adam-rl adam-rl commented Aug 18, 2025

Copy link
Copy Markdown

What this PR does / why we need it: Iterate through all text chunks that need to be emitted as part of the metric text label section before rendering the value and timestamp. Metric rendering was incorrect and unparsable by prometheus.

Please check the following list:

  • Does the affected code have corresponding tests, e.g. unit test, E2E test?
  • Does this change require a documentation update?
  • Does this introduce breaking changes that would require an announcement or bumping the major version?
  • Do all new files have an appropriate license header?

Iterate through all text chunks that need to be emitted as
part of the metric text label section before rendering the
value and timestamp.
@adam-rl
adam-rl requested a review from gautam-rl August 18, 2025 20:30
@adam-rl
adam-rl merged commit f156806 into main Aug 18, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants