Skip to content

Cache llvm-cov tool#34

Merged
frozenspider merged 1 commit into
mainfrom
f/cache-llvm-cov
Dec 26, 2025
Merged

Cache llvm-cov tool#34
frozenspider merged 1 commit into
mainfrom
f/cache-llvm-cov

Conversation

@frozenspider
Copy link
Copy Markdown
Contributor

@frozenspider frozenspider commented Dec 25, 2025 •
edited
Loading

Installing cargo-llvm-cov each time takes a long time and is wasteful. Cache it the same way we cache cargo-deny.

GH Action Time before Time after
Coverage (PR) 166 seconds 38 seconds
Coverage (Push) 87 seconds 18 seconds

@frozenspider frozenspider requested a review from a team as a code owner December 25, 2025 15:50
@Fantoni0
Copy link
Copy Markdown
Contributor

Fantoni0 commented Dec 26, 2025

I think it works, and times are much better.
But recently, I had some issues with the cache of cargo deny NethermindEth/stellar-private-payments#57
We were using an outdated version that was not compatible with newer advisory databases. Even if I updated the version, which cargo deny succeeded, and the new version was not installed.
Could something like this also happen for cache-llvm-cov?

@frozenspider
Copy link
Copy Markdown
Contributor Author

frozenspider commented Dec 26, 2025 •
edited
Loading

@Fantoni0, I've ran into that too. However, the short-term fix is pretty simple - go to Actions > Caches and delete a relevant cache.
I think caching is still worth it even though this might become a problem occasionally.

Fantoni0
Fantoni0 approved these changes Dec 26, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@Fantoni0 Fantoni0 left a comment

Choose a reason for hiding this comment

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

LGTM!

maksimryndin
maksimryndin approved these changes Dec 26, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@maksimryndin maksimryndin left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@frozenspider frozenspider merged commit 9513a3a into main Dec 26, 2025
9 checks passed
@frozenspider frozenspider deleted the f/cache-llvm-cov branch December 26, 2025 12:55
This was referenced Dec 26, 2025
Merged
Merged
frozenspider added a commit that referenced this pull request Dec 26, 2025
@frozenspider
Make caching GitHub actions use separate caches (#35)
ff95ea1 
A follow-up to #34 
Once cache is formed, it's not updated anymore (see
actions/toolkit#505). As such, it doesn't make
sense having the cache shared between unrelated actions.
Made each action using a cache key tied to its tool.
@frozenspider frozenspider mentioned this pull request Dec 27, 2025
Merged
frozenspider added a commit to NethermindEth/zinc-plus that referenced this pull request Dec 29, 2025
@frozenspider
Cache llvm-cov and other tools separately (#60)
d21a3c5 
Adopting NethermindEth/rust-template#34 and
NethermindEth/rust-template#35
ElijahVlasov pushed a commit to NethermindEth/crypto-primitives that referenced this pull request Jan 7, 2026
@frozenspider
Cache llvm-cov and other tools separately (#19)
a4c89bd 
Adopting NethermindEth/rust-template#34 and
NethermindEth/rust-template#35
WiseMrMusa pushed a commit that referenced this pull request Feb 18, 2026
@frozenspider @WiseMrMusa
Cache llvm-cov tool (#34)
518b9e4 
Installing `cargo-llvm-cov` each time takes a long time and is wasteful.
Cache it the same way we cache `cargo-deny`.

| GH Action | Time before | Time after |
| ------------- | ------------- | ------------- |
| Coverage (PR) | [166
seconds](https://github.com/NethermindEth/rust-template/actions/runs/19867825551)
| [38
seconds](https://github.com/NethermindEth/rust-template/actions/runs/20507497399)
|
| Coverage (Push) | [87
seconds](https://github.com/NethermindEth/rust-template/actions/runs/19242367930)
|[ 18
seconds](https://github.com/NethermindEth/rust-template/actions/runs/20507478141)
|
WiseMrMusa pushed a commit that referenced this pull request Feb 18, 2026
@frozenspider @WiseMrMusa
Make caching GitHub actions use separate caches (#35)
e430e56 
A follow-up to #34 
Once cache is formed, it's not updated anymore (see
actions/toolkit#505). As such, it doesn't make
sense having the cache shared between unrelated actions.
Made each action using a cache key tied to its tool.
WiseMrMusa pushed a commit that referenced this pull request Feb 18, 2026
@frozenspider @WiseMrMusa
Cache llvm-cov tool (#34)
c08ad17 
Installing `cargo-llvm-cov` each time takes a long time and is wasteful.
Cache it the same way we cache `cargo-deny`.

| GH Action | Time before | Time after |
| ------------- | ------------- | ------------- |
| Coverage (PR) | [166
seconds](https://github.com/NethermindEth/rust-template/actions/runs/19867825551)
| [38
seconds](https://github.com/NethermindEth/rust-template/actions/runs/20507497399)
|
| Coverage (Push) | [87
seconds](https://github.com/NethermindEth/rust-template/actions/runs/19242367930)
|[ 18
seconds](https://github.com/NethermindEth/rust-template/actions/runs/20507478141)
|
WiseMrMusa pushed a commit that referenced this pull request Feb 18, 2026
@frozenspider @WiseMrMusa
Make caching GitHub actions use separate caches (#35)
47a3b8b 
A follow-up to #34
Once cache is formed, it's not updated anymore (see
actions/toolkit#505). As such, it doesn't make
sense having the cache shared between unrelated actions.
Made each action using a cache key tied to its tool.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Fantoni0 Fantoni0 Fantoni0 approved these changes

@maksimryndin maksimryndin maksimryndin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@frozenspider @Fantoni0 @maksimryndin