-
-
Notifications
You must be signed in to change notification settings - Fork 722
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
docker-slim doesn't work on macos aarch64 (arm on m1) with an aarch64 docker image #213
Comments
Thank you @elliots for opening the issue! Investigating... will update soon. |
@elliots Wonder about the repro steps... You mention downloading zip as a potential way to repro. The mac zip on the github page shouldn't work on M1 because the main app isn't built for darwin arm64 and the sensor is not built for linux arm64. Curious what kind of error message(s) you got trying to use the downloaded zip. The brew installer seems to have a build for M1, but I'm not quite sure how they do the build for their binary package installations and the formula doesn't explicitly set the architecture... Do you mind running |
I tried both brew and zip, its possible it was a different error for the sensor with the zip and I didn't notice (x86 docker-slim will work fine though with rosetta) The brew installed version actually is aarch64, which means that isn't the issue.
But when I run I get
Some other things i just tried:
I then built just docker-slim again myself with :/ My locally built binary:
Not sure what's going on. Maybe something to do with (self) signing? |
That's a lot of useful information... Thank you for confirming the sensor architecture @elliots ! There's a chance that your local Go version is different from the Go version used to compile the sensor. What version of Go do you have installed on your machine? |
|
Thanks @elliots ! Can you also check the Go compiler version for the |
|
Thanks @elliots ! The old(er) Go version theory doesn't seem like the reason... Let's try something different then :) I'll create an M1 build to see if that works. |
@elliots try using this https://downloads.dockerslim.com/releases/1.36.1/dist_mac_m1.zip Let me know if it work or how it fails |
That build worked. |
I've been having the same problem, and alas this build did not work for me, here is my output: ryan@macbookprom1 dist_mac_m1 % ./docker-slim build --target rn:rust --tag rn:rust1 --http-probe=false |
@ryanobjc it's possible |
Oh yes that was it, now it works, thanks :-) |
The volume detection bug should've been addressed already... need to investigate why it's still happening. |
Expected Behavior
Run docker-slim against an aarch64 docker image, it runs.
Actual Behavior
Fails to run... log message:
The issue I think is that there is no arm build of docker-slim-sensor?
Steps to Reproduce the Problem
Specifications
Fix (build it yourself)
The text was updated successfully, but these errors were encountered: