opentelemetry-cpp
opentelemetry-cpp copied to clipboard
Add bzlmod support
Since bazel 6.0, bazel added bzlmod dependency management and tracking system. Currently open telemetry is built by Bazel but is not published to the Bazel Central Regestry, which complicates importing open telemetry.
It would be awesome if open telemetry would be packaged to the BCR.
Is your feature request related to a problem?
Opentelemetry imports dependencies with own opentelemetry_cpp_deps, which is not portable to bzlmod.
Describe the solution you'd like Create MODULE.bazel and publish to the BCR
Describe alternatives you've considered Continue using existing, but it's... Hard...
@auzhva unfortunately we could not offer any official packaging.
While this request makes sense, we (in opentelemetry-cpp) can not provide packaging for every build system.
The best way to address this is to make a contribution to the Bazel Central Registry directly.
This issue is available for anyone to work on. Make sure to reference this issue in your pull request. :sparkles: Thank you for your contribution! :sparkles:
Added help wanted to advertise.
Do not prepare a PR for opentelemetry-cpp, contribute to https://registry.bazel.build/ instead.
This issue was marked as stale due to lack of activity.
:wave: I submitted a PR for the modern bazel dependency management setup here:
I know there was some discussion above about submitting the config to the BCR (which I will do if this PR is merged), but my PR here is still required to modernize the bazel config in this repo.
We could also theoretically remove the old version of doing things in bazel, since right now there is some duplication, but I am initially leaving it in my PR to not affect users on older versions of bazel. I'm happy to remove it though and direct older bazel users to prior releases of this library instead if you'd prefer!
https://github.com/open-telemetry/opentelemetry-cpp/pull/2608
This issue was marked as stale due to lack of activity.
still valid, linked PR should be unblocked when my other prs land