Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add metrics compliance matrix #1956

Merged
merged 19 commits into from
Oct 26, 2021
Merged

Add metrics compliance matrix #1956

merged 19 commits into from
Oct 26, 2021
+88 −3

Conversation

ocelotl
Copy link
Contributor

@ocelotl ocelotl commented Sep 22, 2021

Fixes #1953

@ocelotl ocelotl force-pushed the issue_1953 branch 2 times, most recently from c10277d to de4b764 Compare September 22, 2021 17:53
@ocelotl ocelotl marked this pull request as ready for review September 22, 2021 17:53
@ocelotl ocelotl requested review from a team September 22, 2021 17:53
@ocelotl ocelotl force-pushed the issue_1953 branch 2 times, most recently from 3c909f7 to 92e5fca Compare September 22, 2021 18:10
tigrannajaryan
tigrannajaryan reviewed Sep 22, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang reviewed Sep 24, 2021
View reviewed changes
reyang
reyang previously requested changes Sep 24, 2021
View reviewed changes
Copy link
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

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

Please find my comments.

@reyang reyang added spec:metrics Related to the specification/metrics directory area:miscellaneous For issues that don't match any other area label labels Sep 24, 2021
@reyang reyang added this to the Metrics API/SDK Feature Freeze milestone Sep 24, 2021
@reyang reyang dismissed their stale review September 25, 2021 19:39

All the blocking comments were resolved.

jsuereth
jsuereth reviewed Sep 27, 2021
View reviewed changes
jsuereth
jsuereth reviewed Sep 27, 2021
View reviewed changes
ocelotl and others added 16 commits October 22, 2021 10:49
@ocelotl
Add metrics compliance matrix

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
8ceff4a
@ocelotl
Remove table generator

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
cca58e7
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
f70188c 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
8c4157a 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl
Remove requirement

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
ac5b6b9
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
8559136 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
25a0762 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
ceec30a 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
ddf1b91 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
4d183f2 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
af91da0 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@ocelotl @reyang
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
ff58e3d 
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@jmacd @arminru @ocelotl
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
abc91bc 
Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>
@jmacd @arminru @ocelotl
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
f58dcdd 
Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>
@ocelotl @jmacd
Update spec-compliance-matrix.md

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
97ee7df 
Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>
@ocelotl
Fix document status path

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
6abfbdc
@ocelotl
Add some requested fixes

Verified

This commit was signed with the committer’s verified signature.
ocelotl Diego Hurtado
GPG key ID: F6FC8E499358A322
Verified
Learn about vigilant mode
75625f4
@ocelotl
Copy link
Contributor Author

ocelotl commented Oct 22, 2021

@reyang @jsuereth I have removed all mentions of exemplars, any other lines that had objections and added a disclaimer 👍

Please let me know if you need more changes.

@github-actions github-actions bot removed the Stale label Oct 23, 2021
hdost
hdost approved these changes Oct 23, 2021
View reviewed changes
Copy link

@hdost hdost left a comment

Choose a reason for hiding this comment

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

not that my approval does anything.

@reyang
Copy link
Member

reyang commented Oct 26, 2021

@jsuereth I think this is ready to be merged based on the discussion last week.

@reyang reyang mentioned this pull request Oct 26, 2021
Merged
@reyang
Merge branch 'main' into issue_1953

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
b7d2501
@jsuereth
Merge branch 'main' into issue_1953

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
f1f169f
@jsuereth jsuereth enabled auto-merge (squash) October 26, 2021 16:06
@jsuereth jsuereth merged commit 322cac1 into open-telemetry:main Oct 26, 2021
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
@ocelotl @reyang
@jmacd @arminru @jsuereth
Add metrics compliance matrix (open-telemetry#1956)

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Learn about vigilant mode
47cf640 
* Add metrics compliance matrix

* Remove table generator

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Remove requirement

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Update spec-compliance-matrix.md

Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>

* Update spec-compliance-matrix.md

Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>

* Update spec-compliance-matrix.md

Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>

* Fix document status path

* Add some requested fixes

Co-authored-by: Reiley Yang <reyang@microsoft.com>
Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>
Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>
Co-authored-by: Josh Suereth <joshuasuereth@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmacd jmacd

@tigrannajaryan tigrannajaryan

@arminru arminru

@jsuereth jsuereth

@hdost hdost

@reyang reyang

Assignees

@jmacd jmacd

@arminru arminru

Labels
area:miscellaneous For issues that don't match any other area label spec:metrics Related to the specification/metrics directory
Projects
None yet
Milestone
Metrics API/SDK Feature Freeze
Development

Successfully merging this pull request may close these issues.

Create metrics feature spec compliance matrix
7 participants
@ocelotl @jmacd @reyang @jsuereth @hdost @tigrannajaryan @arminru