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

Can we get ARM64 based docker image? #145

Open
sarathkrishnamurthy opened this issue Aug 13, 2024 · 5 comments · May be fixed by #146
Open

Can we get ARM64 based docker image? #145

sarathkrishnamurthy opened this issue Aug 13, 2024 · 5 comments · May be fixed by #146

Comments

@sarathkrishnamurthy
Copy link

Is it possible to get back the support for ARM based docker images. We have been using this tool for a long time and new we are changing our infra architecture to ARM based. It would be good if we get back the ARM based images.

@aaronpowell
Copy link
Owner

I forgot that was disabled with .NET 7 having a bug but now we're on .NET 8 it shouldn't be a problem.

I've created a PR which adds it back in, and you can find a build here: https://github.com/aaronpowell/httpstatus/pkgs/container/httpstatus-staging

If that container image works on ARM (I don't have an ARM device I can test with) I'll merge the PR

@sarathkrishnamurthy
Copy link
Author

I have tested it M1 mac,
Im getting

$ docker logs e88fc23e6976 Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Teapot.Web, Version=0.1.0.0, Culture=neutral, PublicKeyToken=null'.

@aaronpowell
Copy link
Owner

Sorry, I don't have access to a M1 mac (or any mac for that matter) to test. Can you try building the docker image locally from source and seeing if that works?

@sarathkrishnamurthy
Copy link
Author

I have checked with my colleague who has M1 machice , in which he could able to build and run arm image.

I suspect that the issue is with the docker buildx in github actions, I could see in logs that its trying to pull amd64 base image but not the base image for arm64 base image . (base image i mentioned here is : mcr.microsoft.com/dotnet/sdk:8.0)

@sarathkrishnamurthy
Copy link
Author

Hello @aaronpowell , i have commented a change in PR #146 to fix the above mentioned issue. It would be helpful if you have a look at it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants