Skip to content

chore(ci): update ccache key to use matrix.os instead of runner.os#71

Merged
hariharan-devarajan merged 2 commits into
llnl:developfrom
rayandrew:chore/ci-caching
May 23, 2026
Merged

chore(ci): update ccache key to use matrix.os instead of runner.os#71
hariharan-devarajan merged 2 commits into
llnl:developfrom
rayandrew:chore/ci-caching

Conversation

@rayandrew
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 22, 2026 04:01
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the CI workflow’s ccache cache key to differentiate caches by the configured matrix runner image (matrix.os) rather than the broader platform (runner.os), avoiding cache collisions across different Ubuntu versions while keeping cache reuse via restore keys.

Changes:

  • Switch ccache cache key from ${{ runner.os }} to ${{ matrix.os }}.
  • Switch ccache restore-keys prefixes from ${{ runner.os }} to ${{ matrix.os }}.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rayandrew rayandrew force-pushed the chore/ci-caching branch 2 times, most recently from 6d1fee4 to 5396c83 Compare May 22, 2026 23:45
@hariharan-devarajan hariharan-devarajan merged commit b6ec01b into llnl:develop May 23, 2026
29 checks passed
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.

3 participants