Skip to content

Add caching for tools installed by the runner - #129

Merged
GuillaumeLagrange merged 1 commit into
mainfrom
cod-1336-cache-setup-environment-step-in-the-github-action
Oct 16, 2025
Merged

Add caching for tools installed by the runner#129
GuillaumeLagrange merged 1 commit into
mainfrom
cod-1336-cache-setup-environment-step-in-the-github-action

Conversation

@GuillaumeLagrange

@GuillaumeLagrange GuillaumeLagrange commented Oct 14, 2025

Copy link
Copy Markdown
Contributor
  • For now, limited to apt based system, though mostly tested on ubuntu
  • Set the path to the desired cache location to CODSPEED_SETUP_CACHE_DIR env var or --setup-cache-dir to enable
  • Any tool installed will have the content of its package saved to this directory
  • Before installing any missing tool, first attempt to restore from cache before using apt

This is aimed to be used alongside CI cache to speed up jobs

Fixes #121

@GuillaumeLagrange
GuillaumeLagrange force-pushed the cod-1336-cache-setup-environment-step-in-the-github-action branch 2 times, most recently from 5084704 to 3cc224c Compare October 14, 2025 13:28
@GuillaumeLagrange
GuillaumeLagrange force-pushed the cod-1336-cache-setup-environment-step-in-the-github-action branch from 3cc224c to 0ccaafd Compare October 14, 2025 15:30

@art049 art049 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I let @not-matthias do the final review

Comment thread Cargo.toml
Comment thread src/run/runner/helpers/apt.rs Outdated

@not-matthias not-matthias left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some minor comments on style, looks good otherwise! :)

Comment thread src/run/runner/helpers/apt.rs
Comment thread src/run/runner/valgrind/setup.rs Outdated
Comment thread src/run/runner/helpers/apt.rs Outdated
@GuillaumeLagrange
GuillaumeLagrange force-pushed the cod-1336-cache-setup-environment-step-in-the-github-action branch 2 times, most recently from 8dc7261 to 57ea84d Compare October 16, 2025 14:06

@not-matthias not-matthias left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@GuillaumeLagrange
GuillaumeLagrange force-pushed the cod-1336-cache-setup-environment-step-in-the-github-action branch from 57ea84d to 38b269d Compare October 16, 2025 14:26
@GuillaumeLagrange
GuillaumeLagrange force-pushed the cod-1336-cache-setup-environment-step-in-the-github-action branch from 38b269d to bd4642d Compare October 16, 2025 14:49
@GuillaumeLagrange
GuillaumeLagrange merged commit bd4642d into main Oct 16, 2025
9 checks passed
@GuillaumeLagrange
GuillaumeLagrange deleted the cod-1336-cache-setup-environment-step-in-the-github-action branch October 16, 2025 14:54
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.

valgrind installation is slow (1 - 2 minutes)

3 participants