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

Add ARM64 Support

Open odidev opened this issue 3 years ago • 1 comments
trafficstars

Hi Team,

I'm interested in adding arm64 support in the ci to this repository.

Is your feature request related to a problem? Please describe. This repository doesn’t have arm64 testing support in the GitHub actions ci.

Describe the solution you'd like I have explored the repository and CI being used, arm64 builds can be added in the GitHub actions CI to enable test for arm64.

Describe alternatives you've considered I am working to add arm64 support in GitHub actions to add builds for testing, we can do ARM64 testing in GitHub actions using self-hosted arm64 runners.

If interested, we can get access to ARM H/W through WorksOnArm.

Additional context Do you have any plans for adding the arm64 platform in CI? It will be very helpful if arm64 support is available.

Please share your thoughts on this.

odidev avatar Aug 03 '22 06:08 odidev

@odidev Thanks for the request!

Note that we already ensure the Collector builds on ARM through cross-compilation, so I assume this is about unit tests? Or did you have a different use case in mind?

I have explored the repository and CI being used, arm64 builds can be added in the GitHub actions CI to enable test for arm64.

How can this be done? I tried reading actions/virtual-environments/issues/2552 but found it a bit confusing and I am not sure that this is natively supported by Github. Could you share a bit more about this?

Do you have any plans for adding the arm64 platform in CI? It will be very helpful if arm64 support is available.

AFAIK there are not plans for adding this. We do not have a well-defined policy on what level of testing and support we provide for different architectures, so it's hard to give a general answer that reflects general consensus.

My personal opinion is that adding this would be okay so long as

  1. we track and actively fix or skip unit tests failing on ARM
  2. this does not increase the running time of CI for PRs too much

mx-psi avatar Aug 03 '22 11:08 mx-psi

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

github-actions[bot] avatar Nov 10 '22 03:11 github-actions[bot]

We have a lowish-priority issue over in opentelemetry-php to try building/testing against arm64, and having access to an arm-based self-hosted runner seems like it would help us out.

At the moment, our options seem to be using virtualization, or waiting for github to add arm support to their runners.

brettmc avatar Jan 09 '23 06:01 brettmc

I am in talks with a vendor to get an appliance to run arm tests as a github action runners. I will assign this issue to myself.

atoulme avatar Mar 12 '23 00:03 atoulme

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

github-actions[bot] avatar May 11 '23 03:05 github-actions[bot]

I would love to help get this to happen! How can I help? What would be useful - hardware, cloud credits, or something else? Doing native builds and tests should speed up builds.

dneary avatar May 25 '23 16:05 dneary

I have an Ampere Mt Collins server provided for free by Ampere Computing to support this. It's sitting in my garage and I need help to set it up to use different VMs for different github action workers. Please allow me some time to get to it.

atoulme avatar May 25 '23 16:05 atoulme

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

github-actions[bot] avatar Jul 26 '23 03:07 github-actions[bot]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

github-actions[bot] avatar Sep 25 '23 03:09 github-actions[bot]

I have an Ampere Mt Collins server provided for free by Ampere Computing to support this. It's sitting in my garage and I need help to set it up to use different VMs for different github action workers. Please allow me some time to get to it.

Antoine, we may have an even better option! I believe you already spoke to my colleague Peter - early next week, we should be able to confirm your participation in the actuated pilot - the full details of the pilot, and the link for "How do we get access?" is here: Announcing managed Arm CI for CNCF projects

dneary avatar Nov 17 '23 00:11 dneary

Absolutely, I am working with Peter right now on blogging about our experience. I will read this ASAP.

atoulme avatar Nov 17 '23 00:11 atoulme

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

github-actions[bot] avatar Jan 17 '24 03:01 github-actions[bot]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

github-actions[bot] avatar Mar 19 '24 03:03 github-actions[bot]

@atoulme Should we close this in favor or open-telemetry/opentelemetry-collector/issues/9731?

mx-psi avatar Mar 19 '24 10:03 mx-psi

I think we can keep this issue and make it a blocker to close https://github.com/open-telemetry/opentelemetry-collector/issues/9731.

Currently, we are trying to add an arm64 CI for the collector core repository with https://github.com/open-telemetry/opentelemetry-collector/pull/9584, once that is merged, I can work to add the same level of CI support for contrib.

atoulme avatar Mar 19 '24 22:03 atoulme

Alright, marking as 'on hold' then

mx-psi avatar Mar 20 '24 08:03 mx-psi