Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Whether/how to cache action on self-hosted runner? #145

Closed
TingluoHuang opened this issue Oct 24, 2019 · 5 comments
Closed

Whether/how to cache action on self-hosted runner? #145

TingluoHuang opened this issue Oct 24, 2019 · 5 comments
Assignees

Comments

@TingluoHuang
Copy link
Member

Then runner download the action's tarball/zipball via github api, unzip to disk under _work/_actions/<repo>/<ref>.

The runner won't re-download any action if it's already on disk.

I think we need to re-check this logic before self-hosted runner GA.

Ex:

- uses: actions/checkout@v1

action.yml on disk: _work/_actions/actions/checkout/v1/action.yml

Since action author can repoint their v1 tag to the latest v1 version, ex: v1.0.0 to v1.0.1, the current action download/cache logic will prevent user using self-hosted runner to get latest version on a version channel.

@bryanmacfarlane
Copy link
Member

@ericsciple - we have two choices (1) resolve that ref and cache with the immutable ref or (2) don't cache the action and clear the action cache on the start of a job (hosted wouldn't pay cleanup cost).

When we discussed this awhile ago and launch wasn't resolving, we said it would be done runner side. It sounds like that wasn't done. We should ideally do that but fall back on #2 if that's not practical at this point.

@ericsciple - this is higher priority than checkout v2

@ericsciple
Copy link
Collaborator

I'm planning to clear the cache on job setup. We can add caching later.

@thboop
Copy link
Collaborator

thboop commented Oct 28, 2019

Can we file an issue for caching and link this discussion so we can track it

@ericsciple
Copy link
Collaborator

@thboop ? this is an issue for caching

@ericsciple
Copy link
Collaborator

Closing this issue since the fix is in for self hosted runners. We can open a new issue for tracking later if we want to further enhance this.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants