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

add Docker image support for arm64 chips (Barrie's MacBook Air can test) #1110

Closed
barriebyron opened this issue May 7, 2021 · 7 comments
Closed

Comments

@barriebyron
Copy link
Contributor

barriebyron commented May 7, 2021

per docker/for-mac#5123 (comment)

Docker issue logged and closed because: this is a qemu bug, which is the upstream component we use for running Intel (amd64) containers on M1 (arm64) chips, and is unfortunately not something we control. In general we recommend running arm64 containers on M1 chips because (even ignoring any crashes) they will always be faster and use less memory.

symptom
`➜ starport git:(install-v0.16-docs) docker run -ti -v $HOME/sdh:/home/tendermint -v $PWD:/apps starport/cli:develop app github.com/hello/planet

WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

⭐️ Successfully created a new blockchain 'planet'.
👉 Get started with the following commands:

% cd planet
% starport serve

Documentation: https://docs.starport.network
➜ starport git:(install-v0.16-docs) ✗ docker run -ti -v $HOME/sdh:/home/tendermint -v $PWD:/apps -p 1317:1317 -p 26657:26657 starport/cli:develop serve -p planet

WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
qemu: uncaught target signal 11 (Segmentation fault) - core dumped`

@fadeev
Copy link
Contributor

fadeev commented May 7, 2021

Unassigned @ilgooz because even though this is something we should take care of, it's not a high priority, especially before v0.16.

@ilgooz
Copy link
Member

ilgooz commented May 7, 2021

For feature note:

This requires to add arm64 version of embedded nodetime and protoc binaries.

@faddat
Copy link
Contributor

faddat commented May 25, 2021

Working on this again in #1187

In fact it now builds on my Mac but I'm failing the various tests that we have in Actions.

@aat2703
Copy link

aat2703 commented Feb 7, 2022

Any news on this? This is still not working :/

@ilgooz
Copy link
Member

ilgooz commented Feb 8, 2022

To be able to have this, we first need to be able to compile nodetime (a nodejs runtime embed into Starport) into ARM through zeit/pkg. Right now what have is in this script.

We also need ARM binary for protoc and dart.

@ignite ignite deleted a comment from gitcoinbot Feb 8, 2022
@ignite ignite deleted a comment from gitcoinbot Feb 8, 2022
@ilgooz
Copy link
Member

ilgooz commented Apr 13, 2022

solved by #2247

@ilgooz ilgooz closed this as completed Apr 13, 2022
@ilgooz
Copy link
Member

ilgooz commented Apr 13, 2022

@aat2703 you should now have access to our ARM binaries in the nightly and from our Docker hub images.

We'll include the ARM binaries to the next point release of the CLI after enough testing.

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

No branches or pull requests

5 participants