Skip to content

ci: correct test-elixir cache paths#5826

Merged
max-sixty merged 1 commit intomainfrom
fix/elixir-cache-paths
Apr 26, 2026
Merged

ci: correct test-elixir cache paths#5826
max-sixty merged 1 commit intomainfrom
fix/elixir-cache-paths

Conversation

@prql-bot
Copy link
Copy Markdown
Collaborator

The actions/cache paths in test-elixir.yaml point at directories that don't exist in this repo, so the cache silently never restores or saves. The Elixir job redownloads and recompiles dependencies from scratch on every run.

Evidence

The latest tests run on main (run 24934608816, job 73017930250) shows both cache steps emitting:

Cache not found for input keys: Linux-mix-cache-elixir-deps-...
Cache not found for input keys: Linux-mix-cache-compiled-build-...

…and at job end:

[warning]Path Validation Error: Path(s) specified in the action for caching do(es) not exist, hence no cache is being saved.

actions/cache paths are repo-root relative, not relative to the job's working-directory. The Elixir binding lives at prqlc/bindings/elixir/, so deps and _build resolve to prqlc/bindings/elixir/deps and prqlc/bindings/elixir/_build, not elixir/deps and elixir/_build.

Origin

The mismatch was introduced in #3683 (the directory reorg from bindings/prql-elixirprqlc/bindings/elixir): the working-directory was updated, but the cache path was only partially renamed (prql-elixir/depselixir/deps). The path was wrong before that commit too — prql-elixir/deps was missing the bindings/ prefix — so the cache has likely never worked.

Verification

This is a CI workflow change rather than testable code. After merge, a tests run on main should show:

  • No Path Validation Error warning at the end of the Elixir job.
  • On a second run, the cache steps logging a hit instead of Cache not found, and mix deps.get / mix compile skipping the work.

The actions/cache 'path' is repo-root relative, but the Elixir binding
lives at prqlc/bindings/elixir. Caching elixir/deps and elixir/_build
matches no files, so every run logs 'Path Validation Error' and skips
restore/save.

The mismatch was introduced in #3683 when the directory was renamed; the
working-directory was updated but the cache paths were only partially
renamed.
@max-sixty max-sixty merged commit fb21618 into main Apr 26, 2026
41 checks passed
@max-sixty max-sixty deleted the fix/elixir-cache-paths branch April 26, 2026 08:44
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.

2 participants