Skip to content

CI Use lock file for CircleCI arm64 build #23392

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

Merged
merged 4 commits into from
May 19, 2022

Conversation

lesteve
Copy link
Member

@lesteve lesteve commented May 17, 2022

part of #22425.

This switches the CircleCI arm64 build to lock file.

Main changes:

  • adapts build_test_arm.sh and .circle/config.yml
  • move update_environment_and_lock_files.py to build_tools (before was in build_tools/azure)
  • adds folder to build_metatadata (build_tools/azure vs build_tools/circle)

Copy link
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

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

LGTM, I like how those PRs are progressively remove logics from the CI-specific test scripts to get a more uniform CI environment.

Copy link
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. In general, this is much cleaner. Now that build_tools/azure/update_environments_and_lock_files.py is moved to build_tools/update_environments_and_lock_files.py, there are a few references to update:

git grep -n build_tools/azure/update_environments_and_lock_files
Result
build_tools/azure/debian_atlas_32bit_requirements.txt:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/py38_conda_defaults_openblas_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/py38_conda_forge_openblas_ubuntu_1804_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/pylatest_conda_forge_mkl_linux-64_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/pylatest_conda_forge_mkl_no_coverage_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/pylatest_conda_forge_mkl_osx-64_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/pylatest_conda_mkl_no_openmp_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/pylatest_pip_openblas_pandas_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/pylatest_pip_scipy_dev_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/pypy3_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/azure/ubuntu_atlas_requirements.txt:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/circle/py39_conda_forge_environment.yml:3:# build_tools/azure/update_environments_and_lock_files.py
build_tools/update_environments_and_lock_files.py:4:python build_tools/azure/update_environments_and_lock_files.py
build_tools/update_environments_and_lock_files.py:313:# build_tools/azure/update_environments_and_lock_files.py
build_tools/update_environments_and_lock_files.py:378:# build_tools/azure/update_environments_and_lock_files.py

@glemaitre glemaitre merged commit ef2ef31 into scikit-learn:main May 19, 2022
@glemaitre
Copy link
Member

Thanks @lesteve First time that I look at the system. This is super nice. Great works. The CI will never fail again ;)

lesteve added a commit to lesteve/scikit-learn that referenced this pull request May 19, 2022
@lesteve lesteve deleted the ci-lock-file-circle-arm branch May 19, 2022 16:19
@lesteve
Copy link
Member Author

lesteve commented May 19, 2022

The CI will never fail again ;)

😄 there are still some opportunities for CI failures (e.g. OpenML being down, or setuptools updates for the pip-based builds, and probably more ...), but hopefully the CI will be a bit more stable 😉

glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants