opentelemetry-collector-releases icon indicating copy to clipboard operation
opentelemetry-collector-releases copied to clipboard

Published 20 hours ago • verified publisher icon open-telemetry

distribution for arm7

Open blumamir opened this issue 3 years ago • 7 comments

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

blumamir avatar Apr 27 '22 09:04 blumamir

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.

jpkrohling avatar Apr 27 '22 12:04 jpkrohling

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 🤷‍♂️

blumamir avatar Apr 27 '22 14:04 blumamir

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 avatar May 02 '22 09:05 mx-psi

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

mx-psi avatar Jun 03 '22 07:06 mx-psi

#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?

blumamir avatar Jun 03 '22 07:06 blumamir

#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:

  • [x] add a test step at collector core (here)
  • [ ] Add it to the list or architectures on this repository on goreleaser/configure.go

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

mx-psi avatar Jun 03 '22 08:06 mx-psi

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

blumamir avatar Jun 04 '22 06:06 blumamir