-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
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
Conversation
There was a problem hiding this 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.
There was a problem hiding this 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
Thanks @lesteve First time that I look at the system. This is super nice. Great works. 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 😉 |
part of #22425.
This switches the CircleCI arm64 build to lock file.
Main changes:
build_test_arm.sh
and.circle/config.yml
update_environment_and_lock_files.py
tobuild_tools
(before was inbuild_tools/azure
)folder
tobuild_metatadata
(build_tools/azure
vsbuild_tools/circle
)