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

Added vendor information of Alibaba Cloud #3797

Merged
merged 5 commits into from
Jan 19, 2024

Conversation

steverao
Copy link
Contributor

Added vendor information of Alibaba Cloud

@steverao steverao requested a review from a team as a code owner January 18, 2024 04:09
data/ecosystem/vendors.yaml Outdated Show resolved Hide resolved
@@ -384,3 +384,10 @@
contact: 'https://github.com/adnanrahic'
oss: true
commercial: true
- name: Alibaba Cloud
distribution: true
Copy link
Member

Choose a reason for hiding this comment

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

can you provide evidence for this distribution? Documentation or GitHub repository link

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, we just provided relevant distribution recently and don't release relevant documentation or provide relevant open source codes repository now. But we finished some new instrumentations based on opentelemetry-java-instrumentation. We also start to submit some additional capabilities to upstream. You can see #10258 .

Copy link
Member

Choose a reason for hiding this comment

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

if your documentation is not released yet then I suggest you set distribution to "no" and come back with an update when the documentation is available.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

if your documentation is not released yet then I suggest you set distribution to "no" and come back with an update when the documentation is available.

ok, I adjusted it.

@svrnm svrnm merged commit 9bcc38d into open-telemetry:main Jan 19, 2024
14 checks passed
@svrnm
Copy link
Member

svrnm commented Jan 19, 2024

thank you @steverao

jaydeluca pushed a commit to jaydeluca/opentelemetry.io that referenced this pull request Jan 31, 2024
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.

None yet

2 participants