Skip to content

OM 2.0: Remove MetricPoint concept, and use Sample#2894

Open
dashpole wants to merge 6 commits intoprometheus:mainfrom
dashpole:metricpoint_to_sample
Open

OM 2.0: Remove MetricPoint concept, and use Sample#2894
dashpole wants to merge 6 commits intoprometheus:mainfrom
dashpole:metricpoint_to_sample

Conversation

@dashpole
Copy link
Contributor

Fixes prometheus/OpenMetrics#315

This aligns with the structure in slide 2 of prometheus/OpenMetrics#315 (comment)

Signed-off-by: David Ashpole <dashpole@google.com>
@dashpole dashpole force-pushed the metricpoint_to_sample branch from 2c9d9ae to 65176e7 Compare March 13, 2026 13:59
Signed-off-by: David Ashpole <dashpole@google.com>
@dashpole dashpole force-pushed the metricpoint_to_sample branch from 65176e7 to 4c240ac Compare March 13, 2026 13:59
bwplotka
bwplotka previously approved these changes Mar 13, 2026
Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Reviewed in OM2 WG

Co-authored-by: Bartlomiej Plotka <bwplotka@gmail.com>
Signed-off-by: David Ashpole <dashpole@google.com>
Signed-off-by: David Ashpole <dashpole@google.com>
Signed-off-by: David Ashpole <dashpole@google.com>
@dashpole dashpole force-pushed the metricpoint_to_sample branch from c1aed15 to aad8d1d Compare March 13, 2026 14:50
Signed-off-by: David Ashpole <dashpole@google.com>
@dashpole
Copy link
Contributor Author

@ywwg @bwplotka @krajorama

I addressed all the open comments!

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.

OM 2.0: Clarify the relationship between Metric and MetricPoint

2 participants