actions-setup icon indicating copy to clipboard operation
actions-setup copied to clipboard

use-cache is ignored

Open mariusvniekerk opened this issue 1 year ago • 2 comments

When trying to not use the cache to work around issues with cacheing it still seems to use the cache regardless and fails when doing so?

Run earthly/[email protected]
  with:
    use-cache: false
    version: latest
    prerelease: false
  env:
    CI: true
  
Configured range: latest; allow prerelease: false
Install destination is /home/runner/.earthly
Matched version: v0.[8](https://github.com/XXXXXXXXXXXXXXXXXX/actions/runs/8106015489/job/22155321986#step:4:8).4
Received 1991[8](https://github.com/XXXXXXXXXXXXXXXXXX/actions/runs/8106015489/job/22155321986#step:4:9)14[9](https://github.com/voltrondata/vice-python/actions/runs/8106015489/job/22155321986#step:4:10) of 19918149 ([10](https://github.com/XXXXXXXXXXXXXXXXXX/actions/runs/8106015489/job/22155321986#step:4:11)0.0%), 29.0 MBs/sec
Cache Size: ~19 MB (19918[14](https://github.com/XXXXXXXXXXXXXXXXXX/actions/runs/8106015489/job/22155321986#step:4:15)9 B)
/usr/bin/tar -z -xf /runner/_work/_temp/f6846a4a-fdc3-4528-acbb-ff95f0787[16](https://github.com/XXXXXXXXXXXXXXXXXX/actions/runs/8106015489/job/22155321986#step:4:17)1/cache.tgz -P -C /runner/_work/REPO
Cache restored successfully
Cache restored from key: earthly-cache-Linux-X64-0.8.4
Error: ENOENT: no such file or directory, chmod '/home/runner/.earthly/bin/earthly'

mariusvniekerk avatar Mar 01 '24 03:03 mariusvniekerk

Looking at the code it seems as if core.getInput("use-cache") is never even called

mariusvniekerk avatar Mar 01 '24 04:03 mariusvniekerk

it looks like it's used when deciding to save to the cache under https://github.com/earthly/actions-setup/blob/b61ca413d8100e151c4e517e433a617a402f5051/src/cache-save.ts#L34-L37

maybe something similar can be done under https://github.com/earthly/actions-setup/blob/b61ca413d8100e151c4e517e433a617a402f5051/src/cache-restore.ts#L8 ?

PRs are most welcomed :)

alexcb avatar Mar 01 '24 23:03 alexcb