Skip to content
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

Allow to overwrite ACTIONS_CACHE_URL environment variable #3411

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

enescakir
Copy link
Contributor

The runner script sets the ACTIONS_CACHE_URL using the value from the system connection. Currently, there's no way to override it.

Some community users use custom cache solutions with self-hosted runners. They need to maintain a fork of actions/runner or actions/toolkit, and actions/cache to override the value in their solution. However it leads to significant maintenance work.

If the runner script allows the ACTIONS_CACHE_URL value to be overridden by the CUSTOM_ACTIONS_CACHE_URL environment variable, it makes life a lot easier for the community, includes myself.

An alternative solution could involve allowing the base URL value in actions/toolkit to be overridden. Several individuals have submitted similar PRs to this one actions/toolkit#1695, but they have not received adequate attention from the maintainers.

The runner script sets the `ACTIONS_CACHE_URL` using the value from the
system connection. Currently, there's no way to override it.

Some community users use custom cache solutions with self-hosted
runners. They need to maintain a fork of `actions/runner` or
`actions/toolkit`, and `actions/cache` to override the value in their
solution. However it leads to significant maintenance work.

If the runner script allows the `ACTIONS_CACHE_URL` value to be
overridden by the `CUSTOM_ACTIONS_CACHE_URL` environment variable, it
makes life a lot easier for the community, includes myself.

An alternative solution could involve allowing the base URL value in
`actions/toolkit` to be overridden. Several individuals have submitted
similar PRs to this one actions/toolkit#1695,
but they have not received adequate attention from the maintainers.
@enescakir
Copy link
Contributor Author

Hi @luketomlinson, @ericsciple, @TingluoHuang. Enabling the overwriting the ACTIONS_CACHE_URL variable in actions/runner or actions/toolkit would simplify things for the community. How can we get a comment on one of the PRs?

@enescakir
Copy link
Contributor Author

Hi @igwejk, you've helped me get attention from the maintainers before in another PR. How did you do it? Enabling the overwriting the ACTIONS_CACHE_URL variable in actions/runner or actions/toolkit would simplify things for the community. How can we get a comment on one of the PRs?

@enescakir
Copy link
Contributor Author

Hi @nebuk89, what's your opinion on adding this feature into the actions/runner or actions/toolkit repository?

@kishorviswanathan
Copy link

We would also like to see this merged. We use selfhosted runners and would like to keep the cache offline to reduce internet ingress/egress costs.

@igwejk
Copy link

igwejk commented Sep 13, 2024

Hey @enescakir, let me get back to you later on this.

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

Successfully merging this pull request may close these issues.

3 participants