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.

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

Misc. spec-compliance-matrix updates for Ruby #2642

Conversation

ahayworth
Copy link
Contributor

Changes

- We now specify adding attributes up-front as preferred in our documentation: open-telemetry/opentelemetry-ruby#1291
- We implement all of the attribute limits described in the spec. The
  remainder were implemented in: open-telemetry/opentelemetry-ruby#1292
- We have supported the TextMapPropagator for awhile (v0.17, released 2021-04-22)
- We have supported `OTEL_SERVICE_NAME` for awhile (v1.0.0rc2, released 2021-06-23)
- We support all `OTEL_EXPORTER_{JAEGER,ZIPKIN}` defined in the spec since 2021.
- We definitely suppport the Exporter interface (unclear how we could have hit 1.0 without it).
@ahayworth ahayworth requested review from a team July 2, 2022 19:42
@ahayworth
Copy link
Contributor Author

ahayworth commented Jul 2, 2022

cc @open-telemetry/ruby-approvers @open-telemetry/ruby-maintainers

@tigrannajaryan tigrannajaryan merged commit 819c0c0 into open-telemetry:main Jul 4, 2022
beeme1mr pushed a commit to beeme1mr/opentelemetry-specification that referenced this pull request Aug 31, 2022
- We now specify adding attributes up-front as preferred in our documentation: open-telemetry/opentelemetry-ruby#1291
- We implement all of the attribute limits described in the spec. The
  remainder were implemented in: open-telemetry/opentelemetry-ruby#1292
- We have supported the TextMapPropagator for awhile (v0.17, released 2021-04-22)
- We have supported `OTEL_SERVICE_NAME` for awhile (v1.0.0rc2, released 2021-06-23)
- We support all `OTEL_EXPORTER_{JAEGER,ZIPKIN}` defined in the spec since 2021.
- We definitely suppport the Exporter interface (unclear how we could have hit 1.0 without it).
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.

4 participants