Skip to content

refactor(report)!: Restrict metadata to string entries - #912

Merged
gfieni merged 1 commit into
masterfrom
feat/report-flat-metadata
Aug 24, 2026
Merged

refactor(report)!: Restrict metadata to string entries#912
gfieni merged 1 commit into
masterfrom
feat/report-flat-metadata

Conversation

@gfieni

@gfieni gfieni commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Use canonical metadata entries directly when exporting reports and align report types and tests with the string-only metadata contract.

BREAKING CHANGE: Report metadata values must be strings and nested metadata are not supported anymore.

Use canonical metadata entries directly when exporting reports and align
report types and tests with the string-only metadata contract.

BREAKING CHANGE: Report metadata values must be strings and nested
metadata are not supported anymore.
@gfieni gfieni self-assigned this Aug 24, 2026
@gfieni
gfieni requested a review from a team as a code owner August 24, 2026 10:58
@sonarqubecloud

Copy link
Copy Markdown

@gfieni
gfieni merged commit 814743b into master Aug 24, 2026
8 checks passed
@gfieni
gfieni deleted the feat/report-flat-metadata branch August 24, 2026 11:02
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.26%. Comparing base (5ea6065) to head (dfe70f4).
⚠️ Report is 2 commits behind head on master.

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     #912      +/-   ##
==========================================
- Coverage   80.28%   80.26%   -0.03%     
==========================================
  Files          86       86              
  Lines        3176     3167       -9     
==========================================
- Hits         2550     2542       -8     
+ Misses        626      625       -1     
Files with missing lines Coverage Δ
src/powerapi/database/clickhouse/codecs.py 100.00% <ø> (ø)
src/powerapi/database/influxdb2/codecs.py 84.61% <ø> (ø)
src/powerapi/report/control_report.py 46.15% <100.00%> (-3.85%) ⬇️
src/powerapi/report/formula_report.py 80.00% <100.00%> (-3.34%) ⬇️
src/powerapi/report/hwpc_report.py 66.66% <100.00%> (-2.57%) ⬇️
src/powerapi/report/power_report.py 66.66% <100.00%> (-2.57%) ⬇️
src/powerapi/report/report.py 91.30% <100.00%> (-1.29%) ⬇️
src/powerapi/database/prometheus/codecs.py 84.21% <0.00%> (+4.21%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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