Fix CI: Use a tmpdir SCIE_BASE to force bootstrapping, incorporate Python path into cache key

huonw commented 2 months ago

There were two issues with CI:

It seems like the test that verifies silence on stdout wasn't always doing the required bootstrap. This makes the bootstrap always happen by forcing it to be starting with an empty cache (SCIE_BASE) by setting that to a fresh temp dir.
The ubuntu runner was failing to run the test_pants_source_mode test, because the venv from the restored cache had an symlink to /opt/hostedtoolcache/Python/3.9.18/x64/bin/python (NB version 3.9.18), but the runners have updated to 3.9.19. Fixes #270.

huonw commented 2 months ago

The failure seems to be unrelated. I haven't had time to look into it, yet.

fatal: No names found, cannot describe anything.
/home/runner/work/scie-pants/scie-pants/.scie_pants_dev_cache/clones/pants-2.14.1/pants: line 69: /home/runner/work/scie-pants/scie-pants/.scie_pants_dev_cache/venvs/pants-2.14.1/py39.venv/bin/python: No such file or directory

huonw commented 2 months ago

I've added another fix beyond the original change to set SCIE_BASE, because CI was still failing, due to #270 again.

pantsbuild / scie-pants

Fix CI: Use a tmpdir SCIE_BASE to force bootstrapping, incorporate Python path into cache key #385