-
Notifications
You must be signed in to change notification settings - Fork 487
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
Q: adding an auto-instrumentation from a different monorepo #2238
Comments
In my opinion main benefit of having an instrumentation in this repo would be about visibility and maintainability making it fully open source, Azure Functions team is in the process of adding an instrumentation in this repo, we also have Azure specific resource detectors available here. |
The benefit of having Azure SDK instrumentation here would be that default-distro would provide it and save users from explicitly adding a dependency. In this sense Azure exporter is different because it's an exporter to a specific backend, while Azure SDK instrumentation is similar to AWS , Mongo, or Redis and should be part of the default distro. In case of Java, Azure SDK instrumentation is part of the vanilla otel Java agent - https://github.com/open-telemetry/opentelemetry-java-instrumentation/tree/main/instrumentation/azure-core (as a thin wrapper over So I think it's not a question of a repository, but more of a default distro content. |
With the changes from #2259 it is possible to add the package here in this repo as a vendor component. Keep in mind though: with the future developments that I proposed in #2259 (comment), the autoinstrumentations-node package (and therefore your reason to add the package here) might soon be no-more. The metapackage is part of the issue we're trying to address with the changes in #2259. It simply provides too much incentive for contributors to prefer adding more instrumentations here over hosting it in a seperate repo closer to the community that knows the instrumented package. |
Hiya friends, apologies in advance if this is documented somewhere but I maintain the @azure/opentelemetry-instrumentation-azure-sdk client library and would love to add Azure instrumentation to the auto-instrumentation-node metapackage.
The library is currently hosted in our monorepo where we can absorb any core changes and release independently - are there options or existing libraries that contribute to the auto-instrumentation-node metapackage from a separate repository?
If not, any guidance or ideas y'all can share?
Finally, as a last resort I can move the logic here and deprecate the instrumentation package under the
@azure
namespace but would prefer not to. I'm very interested in adding support for Azure so would love to hear from y'all!The text was updated successfully, but these errors were encountered: