-
-
Notifications
You must be signed in to change notification settings - Fork 2k
fix: define devcontainer platform #5189
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
Conversation
AMD64 version of Go is installed inside the devcontainer but the platform of dokku image is not defined. The platform of the image has to be defined for ARM devices because on these machines ARM version of dokku image is used by default.
|
Worth fixing the golang version to support arm architectures instead? |
|
I will have a look on that. |
|
I keep running into the following error message on M1 when I try to use linux/arm64 platform: |
|
Might be this: dokku/docker/etc/my_init.d/10_dokku_init Line 44 in 82778a0
|
|
It is somehow caused by |
|
Okay, it turns out was caused by |
|
Weird, that definitely is downloadable by me... |
|
With |
|
Is there a better download url? Maybe we need ca-certificates update or something on arm. Also maybe upgrading the focal image works: https://github.com/phusion/baseimage-docker/releases/tag/focal-1.2.0 |
|
I tried which caused and now it seems to be working |
|
Now if I try to open the devcontainer in VSCode I get the following: |
|
Thats strange, we don't open that file in this codebase. What happens if you replace the devcontainer Dockerfile with the upstream |
|
I tried again after stopping and removing everything to cleanup the mess. The currently pushed version seems to be working. The next error I ran into after I tried to push a project: I think this one is caused by a different issue. I tried it with amd64 and arm64 arch images on M1. |
|
You can't use herokuish on arm instances. In the devcontainer, what is the output of |
But the build starts with the following: Because of the warning I would guess it tries to use |
Command seem to work correctly on arm64. Output: https://gist.github.com/nerg4l/bce69372d17cb50a940285bd851d4c8f |
|
Yeah we should disable herokuish on arm64 in the |
|
I redid this in #5392 since I had to switch to an ARM64 machine. No tests with herokuish will work (because herokuish isn't built/released for ARM/ARM64 as most/all of the buildpacks won't work) but thats okay. Thanks for providing the groundwork here, it was super useful! |
AMD64 version of Go is installed inside the devcontainer but the platform of dokku image is not defined. The platform of the image has to be defined for ARM devices because on these machines ARM version of dokku image is used by default.