Description
Description:
When restoring cache, the current cache management system only uses the primary cache key.
GitHub Actions cache allows specifying multiple prefixes, and restoring older caches, to speed up setup even when there has been a dependency change. Cf corresponding documentation
I'd recommend setting this as an opt-in behaviour, toggleable with a new input, because :
- That way, this would not be a breaking change
- Never rebuilding from scratch means stale data could stay forever in cache, which can be dangerous (e.g. a former dependency still present in a virtualenv)
Here's an example of what the configuration could look like (for poetry) :
- uses: actions/setup-python@v4
id: setup-python
with:
python-version: "3.10"
cache: "poetry"
allow-partial-restores: true
Here's a configuration roughly equivalent to what I'm proposing here (the cache key is slightly different, but the idea is the same)
- uses: actions/setup-python@v4
id: setup-python
with:
python-version: "3.10"
- uses: actions/cache@v3
with:
path: /home/runner/.cache/pypoetry/virtualenvs
key: poetry-${{ steps.setup-python.outputs.python-version }}-${{ hashFiles('poetry.lock') }}
restore-keys: |
poetry-${{ steps.setup-python.outputs.python-version }}-${{ hashFiles('poetry.lock') }}
poetry-${{ steps.setup-python.outputs.python-version }}-
We could also expose cache key management to the users through an input parameter, but that seems much more complex and brittle.
Justification:
This change can represent a significant performance improvement on lockfile change, easily several minutes, depending on the project.
Are you willing to submit a PR?
Not at the time, as I have a working workaround (presented above).