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

TravisCI aarch64 job is testing x86_64 not arm64 #16947

Closed
rth opened this issue Jul 26, 2020 · 2 comments
Closed

TravisCI aarch64 job is testing x86_64 not arm64 #16947

rth opened this issue Jul 26, 2020 · 2 comments

Comments

@rth
Copy link
Contributor

rth commented Jul 26, 2020

Unless I'm missing something the aarch64 TravisCI job is actually running a x86_64/amd64 VM, not an ARM one.

This can be seen for instance by the wheels names that are downloaded to run tests in CI,

 Using cached mypy-0.782-cp37-cp37m-manylinux1_x86_64.whl (20.8 MB)

(unlike for e.g. s390x or ppc64le jobs).

The fix would be to specify "arm64" instead of "aarch64" in .travis.yml.

This is due to an unfortunate approach by TravisCI to fall back to amd64 for unsupported architectures, instead of failing (as documented in their docs). The "aarch64" shown in the UI does not seem reliable either, but one can use $TRAVIS_CPU_ARCH environment variable to double check.

Maybe they renamed this arch at some point, not sure.

Discovered while trying to add a similar setup in scikit-learn/scikit-learn#17996

cc @mattip

@mattip
Copy link
Member

mattip commented Jul 26, 2020

Indeed. You can see in a build log from a recent PR (expand the "Build system information" to see line 39) that the machine architecture is linux/amd64, even though the architecture name is aarch64. Thanks!

@rth
Copy link
Contributor Author

rth commented Jul 28, 2020

Fixed in #16957, thanks!

@rth rth closed this as completed Jul 28, 2020
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 a pull request may close this issue.

2 participants