-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cache miss for existing keys when merging PRs #1561
Comments
I seem to be running into the same issue. As I tried with a few different actions/cache version, I ended up in a situation where it wouldn't save the cache because it already exists, but there are no caches listed on the caches page for my repo. |
Ah, I see this in GitHub's documentation: Caching dependencies to speed up workflows
I've decided to use a 3rd party action which allows this to work correctly, but by downloading artifacts from other workflows instead of accessing cache: https://github.com/dawidd6/action-download-artifact This looks like the path others took as well, but this is a serious limitation of the cache feature. I understand not being able to access child or sibling branches, but it's a glaring issue if a base branch cannot access the cache of the source merged into it. |
For me this is all on the same branch, so maybe I'm running into a separate issue. (edit: apparently the issue is that I was using different file paths between my restore and save steps) |
Thanks to this comment which saved me some headache: actions#1491 (comment) related to actions#1561. related to actions#1491. related to actions#1426.
I have a workflow that is saving a cached file from the head branch to the base branch on a pull request merge. The file keys exists on the head branch, but when it attempts to find it using a
restore-keys
basic path, it is failing to hit. I see the key exists in my GitHub cache for the head branch, so not sure why it's showing as missing.Here is an example workflow:
The text was updated successfully, but these errors were encountered: