Skip to content
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

feat: add support for linux arm64 #178

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Johnee
Copy link

@Johnee Johnee commented Jan 23, 2023

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Snyk offers a pre-built binary for Linux Arm64. One of the use-cases for this is running Linux containers on M1/M2 Apple laptops, specifically with 'linux/arm64/v8' architecture. This change allows the Snyk Maven plugin to detect the appropriate architecture and download the correct binary.

Without this change, the error received is #17 13.87 [ERROR] qemu-x86_64: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory, while uname -m shows aarch64.

I am not fully sure about the state of GitHub Actions tests, they seem to be broken for the past two months. The surefire tests are passing, and the plugin has been tested to work correctly in a local test context.

Where should the reviewer start?

How should this be manually tested?

Any background context you want to provide?

What are the relevant tickets?

Screenshots

Additional questions

Snyk offers pre-built binary for Linux Arm64. One of the usecases for
this is running Linux containers on M1/M2 Apple laptops, specifically
with 'linux/arm64/v8' arhitecture. This change allows the Snyk Maven
plugin to detect the appropriate architecture and download the
correct binary.
@Johnee Johnee requested a review from a team as a code owner January 23, 2023 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant