docs icon indicating copy to clipboard operation
docs copied to clipboard

Update OM 2.0 with proposal#43 Created Timestamp syntax

Open ArthurSens opened this issue 8 months ago • 3 comments

Updates OpenMetrics 2.0 with our discussions in the working group and https://github.com/prometheus/proposals/pull/43

cc @Maniktherana

ArthurSens avatar Apr 28 '25 19:04 ArthurSens

Approved with some small corrections. Also as per WG decision, you'll need another approval.

Thanks for clarifying about exemplars :)

ArthurSens avatar Jun 06 '25 20:06 ArthurSens

I wonder if @ct needs to be prohibited explicitly for gauges, gauge histograms and alike. The ABNF does allow it, so syntax doesn't restrict, needs to be a semantic rule.

krajorama avatar Jun 26 '25 07:06 krajorama

Comments addressed, thanks for the review!

I wonder if @ct needs to be prohibited explicitly for gauges, gauge histograms and alike. The ABNF does allow it, so syntax doesn't restrict, needs to be a semantic rule.

Do we mean that we need to be explicit in the Gauge/Gauge Histogram definition that they do not have created timestamps? The current definition doesn't say that created timestamps exist for them, so we can rely on the implicit information that those types don't have CT...?

ArthurSens avatar Jun 26 '25 21:06 ArthurSens

@krajorama, do you want to take another look or am I ok to merge?

ArthurSens avatar Jun 30 '25 13:06 ArthurSens