-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 integrations #3278
Update integrations #3278
Conversation
Signed-off-by: svrnm <[email protected]>
Signed-off-by: svrnm <[email protected]>
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.
Awesome start
We should add integrations as a link on the homepage too - https://opentelemetry.io/ |
Not directly related to this PR but we may need to rethink the registry, adopters, and integrations pages a bit. Loose overlap that maybe could be combined? |
Yes, we need to rethink a lot of that. for example if you look into the vendors there are 90% backends, but there are also at least two vendors of pipelines and probably some other non-backends, which may count as integrations or "something else" |
not with this PR. |
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.
LGTM, some suggestions inline
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.
Made some suggestions to streamline a little bit and reduce marketing overtones. (Although perhaps that's intended?)
Co-authored-by: Fabrizio Ferri-Benedetti <[email protected]>
thank you @mhausenblas & @theletterf, I have applied most of your suggestions :-) |
Signed-off-by: svrnm <[email protected]>
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.
Thank you!
Signed-off-by: svrnm <[email protected]>
5b46734 includes a few additional words for the documentation index: https://deploy-preview-3278--opentelemetry.netlify.app/docs/ |
Would this be a topic for the communications sig or another one? Happy to join and brainstorm or talk through potential re-orgs |
Sig Comms, yes. We have had our share of discussions on this topic already, if you review some old issues (like #2178) you will get some more context. This PR is probably not the best place to discuss this. Feel free to raise a new meta issue, we can link to existing issues from there and have a consolidated discussion. I may have the one or the other opinion as well ;-) |
Small follow up PR - #3306 |
This is going to address some of #3221 requirements, cc @cartersocha