search

open-telemetry / opentelemetry-collector-releases

OpenTelemetry Collector Official Releases
https://opentelemetry.io
Apache License 2.0
229 stars 144 forks source link

distribution for arm7 #105

Open blumamir opened 2 years ago

blumamir commented 2 years ago

Hi, I want to use the collector on Raspberry PI with docker, and was not able to find a relevant image in dockerhub to use. Looking at some old issues and PRs it seems to be removed intentionally. Was wondering if there are plans to support it any time soon

jpkrohling commented 2 years ago

I think we had it at some time but had to remove it due to build issues with a component. Would you be able to try to build the collector for arm7 and report back? If it works and is part of the continuous testing for the core and contrib repos, we could certainly add an official build for it.

blumamir commented 2 years ago

I think we had it at some time but had to remove it due to build issues with a component. Would you be able to try to build the collector for arm7 and report back? If it works and is part of the continuous testing for the core and contrib repos, we could certainly add an official build for it.

Thanks for the reply Unfortunately, I have no background in GO and have too much on my plate right now to familiarize myself with the build scripts of the collector. I will keep this issue here, and maybe someone will pick it up 🤷‍♂️

mx-psi commented 2 years ago

This is blocked by open-telemetry/opentelemetry-collector-contrib#8544, it should be easy to fix if you want to try, even if you don't know Go! For comparison this is what it took on the core distro: open-telemetry/opentelemetry-collector#5142

mx-psi commented 2 years ago

#144 will revert the PR that removed this (nope, I got confused)

blumamir commented 2 years ago

144 will revert the PR that removed this

Thanks for the update. This PR seems to be adding 386, will it also support raspberry pi that is running arm v7?

mx-psi commented 2 years ago

144 will revert the PR that removed this

Thanks for the update. This PR seems to be adding 386, will it also support raspberry pi that is running arm v7?

Oh, of course, I got confused by the "Looking at some old issues and PRs it seems to be removed intentionally." 🤦 This will add back support for x86 32 bits, sorry to get your hopes up.

Support for armv7 has not been available since #43 because at some point contrib was not buildable on that architecture. We currently build on armv7 on the contrib repo (see here) so we could add it here. The steps to do it would be to:

Both are 1-line PRs and I see no reason to reject them, so you are welcome to contribute :)

blumamir commented 2 years ago

Both are 1-line PRs and I see no reason to reject them, so you are welcome to contribute :)

Created PR in #146 . Thanks for all the help and guidness