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 workaround for Arm based MACs #272

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

WadeBarnes
Copy link
Member

  • Auto-detect Arm based MAC and set DOCKER_DEFAULT_PLATFORM=linux/amd64 to ensure docker uses amd64 emulation and uses the available base images.
  • This is required since there are no Arm based base images for the project currently.

- Auto-detect Arm based MAC and set `DOCKER_DEFAULT_PLATFORM=linux/amd64` to ensure docker uses amd64 emulation and uses the available base images.
- This is required since there are no Arm based base images for the project currently.

Signed-off-by: Wade Barnes <wade@neoterictech.ca>
@WadeBarnes WadeBarnes requested a review from swcurran July 19, 2023 12:42
@WadeBarnes
Copy link
Member Author

I don't have an Arm based MAC to test this on, so it would be nice if the changes could be tested by someone that does. Thanks

@swcurran
Copy link
Contributor

@andrewwhitehead — can you try this? Basically, on the current release, run:

./manage build; ./manage run and it should fail. Get this PR and rerun and it should work.

I might get to it, but I use my Mac as a display machine only — all my development tools are on another machine. I’d rather not add a pile of things to this one...

@swcurran swcurran requested a review from andrewwhitehead July 19, 2023 17:51
@esune
Copy link
Member

esune commented Aug 8, 2023

Tested on M1 Mac, works as expected.

@esune esune merged commit 08787d6 into bcgov:main Aug 8, 2023
WadeBarnes pushed a commit to WadeBarnes/von-network that referenced this pull request Aug 28, 2024
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 this pull request may close these issues.

3 participants