Pin macos python versions in CI to fix mismatch between builder and test runner by scottvanta · Pull Request #8559 · osquery/osquery
Changes:
- Pin a specific Python version (3.13.1) when installing python on the macos builder and the test_older_macos runner
Motivation:
- Fix the test_older_macos CI tests
- Failures are of the form "Could not start process at /Users/runner/hostedtoolcache/Python/3.13.1/x64/bin/python3" (e.g. this run)
- The macos-13 runner image was recently updated to have Python 3.13.2 but the other macos runners are still on Python 3.13.1
- Notably, the "Setup python" step in the test_older_macos run shows "Successfully set up CPython (3.13.2)", which mismatches the version (3.13.1) that gets set up in the "Setup python" step of the build_macos (macos-14, x86_64) build that builds the macos_tests_Debug/Release artifacts uses in test_older_macos
Test plan:
- CI
- (I've validated that this change fixes the test_older_macos failures on a local fork with similar CI setup)
@scottvanta I kicked off the CI. Will be happy to merge once CI completes and you sign the CLA. Thank you!
Thanks! I've been working through the steps of getting the corporate CLA set up, hopefully should have an update soon!
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Scott for the PR and further commentary! I think it makes sense to pin it like you said.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters