-
Notifications
You must be signed in to change notification settings - Fork 893
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
[semantic conventions] add alibaba cloud as a cloud provider #1831
[semantic conventions] add alibaba cloud as a cloud provider #1831
Conversation
e6b1fe8
to
5f0cc9e
Compare
Thank you for the PR! EDIT: See here: opentelemetry-specification/semantic_conventions/trace/faas.yaml Lines 153 to 171 in 3422558
(note that there is an open tooling issue for sharing enum definitions: open-telemetry/build-tools#32, but I doesn't look like it will be resolved in the foreseeable future) |
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.
Thanks! Please also add an entry in CHANGELOG.md for your addition.
853cb80
to
2f9fbad
Compare
Just pushed to trigger unresponsive EasyCLA to re-check. |
…lemetry#1831) Co-authored-by: Armin Ruech <[email protected]> Co-authored-by: Carlos Alberto Cortez <[email protected]>
Changes
Add Alibaba Cloud as one of the cloud providers. So that SDK implementations with the generated definition can start working on detector support, etc.
Alibaba Cloud has ~10% of the global market share, ranking third in the world and first in Asia-Pacific.
Related issues open-telemetry/opentelemetry-js#2378