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

Update Grafana Labs distribution status #3553

Merged
merged 3 commits into from
Nov 21, 2023

Conversation

tpaschalis
Copy link
Member

@tpaschalis tpaschalis commented Nov 16, 2023

Hi! This PR marks Grafana Labs as a vendor offering a distribution on the OpenTelemetry vendor list.

Link to your distribution, if applicable:
https://github.com/grafana/agent

In the past year, we've been working on enabling Grafana Agent to work natively with OTLP, and offering multiple OTel collector components as a distribution (example).

We're also maintaining two instrumentation distros in:
https://github.com/grafana/grafana-opentelemetry-java
https://github.com/grafana/grafana-opentelemetry-dotnet

Link that proves that your offering is open source, if applicable. An open source distribution does not qualify your offering to be marked “open source”.

The mentioned projects all use the Apache License 2.0
https://github.com/grafana/agent/blob/main/LICENSE
https://github.com/grafana/grafana-opentelemetry-java/blob/main/LICENSE
https://github.com/grafana/grafana-opentelemetry-dotnet/blob/main/LICENSE

GitHub handle or email address as a point of contact so that we can reach out in case we have question
@jpkrohling
@gouthamve
@richardqlam
@rfratto
@matt-hensley

@tpaschalis tpaschalis requested a review from a team November 16, 2023 15:42
@@ -91,7 +91,7 @@
oss: false
commercial: true
- name: Grafana Labs
distribution: false
distribution: true
nativeOTLP: true
url: 'https://grafana.com/oss/opentelemetry/'
contact: ''
Copy link
Member

Choose a reason for hiding this comment

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

While updating this, can you please also list a contact, thanks!

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, adding @jpkrohling's github account here.

Copy link
Member

Choose a reason for hiding this comment

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

@jpkrohling: please approve this PR so I know you are OK with that:)

Copy link
Member

Choose a reason for hiding this comment

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

Done.

Signed-off-by: Paschalis Tsilias <[email protected]>
@tpaschalis tpaschalis requested a review from svrnm November 16, 2023 17:36
@svrnm svrnm requested a review from jpkrohling November 16, 2023 18:46
@svrnm
Copy link
Member

svrnm commented Nov 20, 2023

@tpaschalis you created this fork without the permissions of maintainers to push commits to it, that way I am not able to update the branch to be in sync with the main branch. Please take care that the PR is in sync such that we can merge it

@tpaschalis
Copy link
Member Author

@svrnm apologies, I read that in the contribution guide but missed the step.

I've updated this branch with main.

@svrnm svrnm merged commit e8fc245 into open-telemetry:main Nov 21, 2023
14 checks passed
@svrnm
Copy link
Member

svrnm commented Nov 21, 2023

@svrnm apologies, I read that in the contribution guide but missed the step.

I've updated this branch with main.

No worries, as a rule of thumb, do not fork a repo within an organization if you want to do an upstream PR, because they default to now providing write permissions

@tpaschalis tpaschalis deleted the update-grafanalabs-distribution branch November 21, 2023 13:11
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.

3 participants