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

Closes #601: Force Docker containers to use amd64 for now. #603

Merged
merged 1 commit into from
Nov 18, 2022

Conversation

joeparsons
Copy link
Member

@joeparsons joeparsons commented Nov 16, 2022

Alternative fix/workaround for local build issues affecting M1 Macs (#601)

@joeparsons joeparsons self-assigned this Nov 16, 2022
@joeparsons joeparsons marked this pull request as ready for review November 16, 2022 23:39
@joeparsons joeparsons requested a review from a team November 16, 2022 23:39
@joeparsons joeparsons added the enhancement New feature or request label Nov 16, 2022
@joeparsons joeparsons changed the title Closes #601: Force Docker images/containers to use amd64 for now. Closes #601: Force Docker containers to use amd64 for now. Nov 16, 2022
@mmunro-ltrr
Copy link
Member

I rebuilt with the local Intel version of Docker just to make sure it recognized the modified Dockerfile.

@trackleft
Copy link
Member

This only worked for me when I upgraded to the latest version of docker desktop 4.14.1

As expected I received the warning

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

@trackleft
Copy link
Member

We can merge this

@trackleft trackleft merged commit 999ba03 into main Nov 18, 2022
@trackleft trackleft deleted the issue/601-Dockerfile branch November 18, 2022 20:38
@trackleft trackleft mentioned this pull request Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants