-
Notifications
You must be signed in to change notification settings - Fork 888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move metrics SDK spec to Mixed #2304
Move metrics SDK spec to Mixed #2304
Conversation
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
We should revisit this PR once #2317 is merged. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Note that #2072 appears to be still blocking this PR, unfortunately. In #2072 (comment) I have a proposal that would also let us close #2232, I think. Please give #2072 and #2232 your attention, especially @mateuszrzeszutek, @legendecas, @jack-berg, @jsuereth, @reyang thanks. |
I think that having some sort of resolution of #2314 is important for stabilization. My read is that there's consensus that delta backends actually want cumulative non-monotonic sums instead of delta non-monotonic sums for sync / async up down counter instruments. Stabilizing without some resolution would create trouble for these backends without benefiting anyone (as far as I can tell). |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
re-approval after more recent PRs are merged.
This PR is blocked by a CI issue which is being addressed in #2426. |
Following up from the specification call today: this is ready to merge |
The Metrics SDK specification stabilization progress is tracked by https://github.com/orgs/open-telemetry/projects/9.
During the past 4 weeks, we haven't seen outstanding new issues during the weekly triage meeting, and all of the known blocking issues have been properly resolved.
Changes