[CI] Use a Stable Cache Key to prevent warnings in gh-actions #456
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
In some of the gh-action test runs, I have observed the following warnings for the model and dataset cache not being saved correctly:
The issue stems from setting the cache key in an uncertain manner. When tests get updated or modified, a new key will be assigned and the previous cache will get invalidated.
In the updated key, we use a stable prefix
hf-cache-${{ runner.os }}-, which cleanly separates caches by operating system. The addition of${{ hashFiles('pyproject.toml') }}ensures the cache is only invalidated when your dependency configuration changes. This approach also enables partial key matching through restore keys, allowing gh-actions to fall back to compatible caches when an exact match can't be found.Could you please review?
cc: @davidberenstein1957
Related Issue
Fixes #(issue number)
Type of Change
How Has This Been Tested?
Checklist
Additional Notes