-
Notifications
You must be signed in to change notification settings - Fork 370
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
Add cache-prefixing #1169
base: main
Are you sure you want to change the base?
Add cache-prefixing #1169
Conversation
Co-authored-by: ccoVeille <[email protected]>
Co-authored-by: ccoVeille <[email protected]>
I am hitting similar issue but this time it is because I have couple of runners working on the same VM. In this situation the user (under which the first runner operates) creates the folder and file and remains owner preventing other runners (running under different context) to have access to the tmp folder. |
It's been a while so the full reasoning escapes me. I remember seeing your PR and the reasoning why I created a separate one was because:
|
Description:
This changes allow the state saved to the global action cache to be stored with a prefix. Useful as
_state
is not a unique name and multiple workflows could save the cache as_state
. Also useful as you currently cannot use this action twice for independent runs as the runs would pick up the same state. With this change, it is possible to have one run close all PRs with labelx
after 7 days and another run close all PRs except those with labelx
after 10 days.Also updated the README.md to show proper permissions needed.
Related issue:
#1137
Please also look into merging PR #1152 as this isn't necessarily useful if caching isn't 100% fixed.
Check list: