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

Cache eviction feature for actions/setup-node #10311

Closed
1 of 14 tasks
karlhorky opened this issue Jul 19, 2024 · 2 comments
Closed
1 of 14 tasks

Cache eviction feature for actions/setup-node #10311

karlhorky opened this issue Jul 19, 2024 · 2 comments
Assignees

Comments

@karlhorky
Copy link

karlhorky commented Jul 19, 2024

To be clear, this is not a tool request - it's a feature request for runner-images, requested by @aparnajyothi-y of the actions/setup-node team

Tool name

actions/setup-node

Tool license

MIT

Add or update?

  • Add
  • Update

Desired version

--

Approximate size

No response

Brief description of tool

Cache eviction feature for @aparnajyothi-y and the rest of the actions/setup-node team, as requested here:

This will enable purging of old cached LTS versions to avoid the problem that node-version: 'lts/*' installs an outdated version, as described in the issue:

URL for tool's homepage

No response

Provide a basic test case to validate the tool's functionality.

No response

Platforms where you need the tool

  • Azure DevOps
  • GitHub Actions

Runner images where you need the tool

  • Ubuntu 20.04
  • Ubuntu 22.04
  • Ubuntu 24.04
  • macOS 12
  • macOS 13
  • macOS 13 Arm64
  • macOS 14
  • macOS 14 Arm64
  • Windows Server 2019
  • Windows Server 2022

Can this tool be installed during the build?

No response

Tool installation time in runtime

No response

Are you willing to submit a PR?

No response

@karlhorky karlhorky changed the title Cache Eviction for actions/setup-node Cache eviction feature for actions/setup-node Jul 19, 2024
@hemanthmanga
Copy link
Contributor

Hi @karlhorky As the runner images team, we believe cache eviction should be handled through tasks, not the runner itself.
Please look into the comment

@karlhorky
Copy link
Author

Ok, that's unfortunate for users.

This common lts/* config for actions/setup-node is broken (does not retrieve latest LTS version, even days after release), and if actions/runner-images won't build anything to fix it, it will stay broken:

      - name: Use Node.js
        uses: actions/setup-node@v4
        with:
          node-version: 'lts/*'
          cache: 'pnpm'

Source: actions/setup-node#940 (comment)

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

No branches or pull requests

3 participants