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

FaaS conventions: make faas.document.time and faas.time attributes recommended #2627

Merged
merged 3 commits into from
Jun 28, 2022

Conversation

lmolkova
Copy link
Contributor

Fixes #2599

Changes

faas.document.time and faas.time levels are changed from requirement to recommended as span start time can be used instead when high precision is not so important.

@lmolkova lmolkova requested review from a team June 21, 2022 21:03
@lmolkova
Copy link
Contributor Author

@open-telemetry/specs-trace-approvers can someone please review?

@arminru arminru added area:semantic-conventions Related to semantic conventions spec:trace Related to the specification/trace directory labels Jun 28, 2022
@arminru arminru merged commit 42266f5 into open-telemetry:main Jun 28, 2022
beeme1mr pushed a commit to beeme1mr/opentelemetry-specification that referenced this pull request Aug 31, 2022
beeme1mr pushed a commit to beeme1mr/opentelemetry-specification that referenced this pull request Aug 31, 2022
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:semantic-conventions Related to semantic conventions spec:trace Related to the specification/trace directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Faas trace attributes: requirement levels need revision
4 participants