fix: yarn cache problem introduced in #21046 #21105
Merged
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
#21046 introduced a problem because we were restoring cache from
dependency-cache-{{ checksum "/tmp/YARN_VERSION" }}-{{ checksum "yarn.lock" }}
but saving cache to
dependency-cache-v1-{{ checksum "yarn.lock" }}
Thus, nothing ever came back from the cache
The result of this was in no way critical, just inefficient.
Manual testing steps
See that the CI step
prep-deps
completes successfully and restores from cacheRelated issues
Fixes problem introduced in #21046
Pre-merge author checklist
Pre-merge reviewer checklist