Skip to content

refactor(database/prometheus): Encode missing tags as empty labels - #916

Merged
gfieni merged 1 commit into
masterfrom
fix/prometheus-tag-unknown
Aug 25, 2026
Merged

refactor(database/prometheus): Encode missing tags as empty labels#916
gfieni merged 1 commit into
masterfrom
fix/prometheus-tag-unknown

Conversation

@gfieni

@gfieni gfieni commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Use an empty label value when a configured tag is absent from PowerReport metadata instead of assigning the synthetic "unknown" value.

This aligns missing tags with Prometheus semantics and avoids conflating absent metadata with a legitimate "unknown" tag value.

Use an empty label value when a configured tag is absent from
PowerReport metadata instead of assigning the synthetic "unknown" value.

This aligns missing tags with Prometheus semantics and avoids conflating
absent metadata with a legitimate "unknown" tag value.
@gfieni gfieni self-assigned this Aug 25, 2026
@gfieni
gfieni requested a review from a team as a code owner August 25, 2026 11:53
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.26%. Comparing base (932bd7f) to head (6b3d0fa).

Files with missing lines Patch % Lines
src/powerapi/database/prometheus/codecs.py 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #916   +/-   ##
=======================================
  Coverage   80.26%   80.26%           
=======================================
  Files          86       86           
  Lines        3167     3167           
=======================================
  Hits         2542     2542           
  Misses        625      625           
Files with missing lines Coverage Δ
src/powerapi/database/prometheus/codecs.py 84.21% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gfieni
gfieni merged commit 84e5ae7 into master Aug 25, 2026
10 checks passed
@gfieni
gfieni deleted the fix/prometheus-tag-unknown branch August 25, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant