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

Update arangodb docs #2359

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nerpaula
Copy link

Added note about macOS/ARM and how to override the default chosen processor architecture.

@nerpaula nerpaula marked this pull request as ready for review August 10, 2023 10:41
Copy link
Contributor

@dothebart dothebart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Paula.

@tianon why is it that these macs rather choose the incomplete Rosetta x86 emulation than native ARM docker containers?

@tianon
Copy link
Member

tianon commented Dec 14, 2023

Yeah, that sounds to me like a solid bug elsewhere 🤔

@nerpaula can you give more details about what your environment looked like when you ran into this?

@nerpaula
Copy link
Author

@tianon this issue was reported here: arangodb/arangodb-docker#114

@yosifkit
Copy link
Member

Considering that the problem is caused by configuration on the host (DOCKER_DEFAULT_PLATFORM=linux/amd64), I don't think this should be called out on a specific image's docs. The users' config/environment is overriding the best choice. It is unfortunate that docker doesn't warn the user when DOCKER_DEFAULT_PLATFORM is used.

@dothebart
Copy link
Contributor

dothebart commented Dec 20, 2023

shouldn't it do that? Since I think we saw more than one mac user run into this trap.
While most other users in this situation will probably only experience performance degradation of the emulation, only containers that use non supported processor optimizations will unveil this.
Since docker is now our recommended way to run ArangoDB on mac, I'd like to have a good solution for this. Is there a way to say you need AVX2 for docker containers etc?

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.

4 participants