actions / actions/cache

`Cache save failed.` should include the name of the cache and the failure class

Open
#1,213 2 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
5.6k
Forks
1.6k
PR merge metrics
No merged PRs in 30d

Description

https://github.com/check-spelling/almalinux-wiki/actions/runs/5655212720

Annotations

1 warning
Check Spelling
Cache save failed.

  1. Users shouldn't have to click the not particularly obvious link to determine which cache failed (especially given that the link doesn't actually expand the log and getting to the thing once you're at the log is a PITA).
  2. It would be nice if already exists was included in the annotation

For reference, the log does have the information available, so including it in the ::warning ::output would not be hard.

2023-07-25T09:46:19.6142161Z [command]/usr/bin/tar --posix -cf cache.tzst --exclude cache.tzst -P -C /home/runner/work/almalinux-wiki/almalinux-wiki --files-from manifest.txt --use-compress-program zstdmt
2023-07-25T09:46:19.8054484Z Failed to save: Unable to reserve cache with key check-spelling-dictionaries-a:ce53902f1052602f8f1fe328d65ace5561a1e8c1-29b2bbcc31edb0dd98b0a3ab90fa2e1d8f6d67b2, another job may be creating this cache. More details: Cache already exists. Scope: refs/heads/spell-check-with-spelling, Key: check-spelling-dictionaries-a:ce53902f1052602f8f1fe328d65ace5561a1e8c1-29b2bbcc31edb0dd98b0a3ab90fa2e1d8f6d67b2, Version: 3c9138c5df6ff04e024d625faac2cacce774cfddeb89e911df030b9c6871f016
2023-07-25T09:46:19.8100252Z ##[warning]Cache save failed.
Expected results
::warning ::Failed to save cache `check-spelling-dictionaries-a:ce53902f1052602f8f1fe328d65ace5561a1e8c1-29b2bbcc31edb0dd98b0a3ab90fa2e1d8f6d67b2` -- cache already exists

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Use the linked Actions run and its warning text as the behavioral reference; no source file or test is named, so first locate the TypeScript code that emits the cache-save warning. Done means the warning identifies the cache and includes the failure class, including the “already exists” case.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
ci-cd
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.