`Cache save failed.` should include the name of the cache and the failure class
Open
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.
- 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).
- It would be nice if
already existswas 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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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