Skip to content

Experiment with GHA caching#83

Merged
teunbrand merged 3 commits intoposit-dev:mainfrom
teunbrand:caching_experiment
Jan 22, 2026
Merged

Experiment with GHA caching#83
teunbrand merged 3 commits intoposit-dev:mainfrom
teunbrand:caching_experiment

Conversation

@teunbrand
Copy link
Collaborator

Current GHA takes a long time, maybe we can shave off a good chunk?

Copy link
Collaborator

@thomasp85 thomasp85 left a comment

Choose a reason for hiding this comment

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

Maybe add a new commit here to see if it picks up the cache? If it does you can merge at your leisure

@teunbrand
Copy link
Collaborator Author

teunbrand commented Jan 22, 2026

Just noting down results of run 1:

build: 14m49s
publish: 22m25s
check: 22m15s

Restarting the jobs manually for run 2.

@teunbrand
Copy link
Collaborator Author

Second run has:

build: 14m48s
publish: 21m4s
check: 22m0s

This is not a noteworthy improvement.
Let me try some other GHA actions custom to rust

@teunbrand
Copy link
Collaborator Author

teunbrand commented Jan 22, 2026

Times for the uncached run with the new step:

build: 15m25s
publish: 21m56s
check: 23m0s

For the cached run:

build: 3m33s
publish: 23m38s
check: 22m46s

@thomasp85
Copy link
Collaborator

it seems the build at least was succesful

@thomasp85
Copy link
Collaborator

but even with the cache the Jupyter kernel needs a full rebuild...

@teunbrand
Copy link
Collaborator Author

teunbrand commented Jan 22, 2026

Yes, I'm going to try to give each job individual keys next. So they don't all need to build from the same cache, but each can cache its own stuff.

@teunbrand
Copy link
Collaborator Author

Uncached run:

build: 14m
publish: 21m
check: 23m

@teunbrand
Copy link
Collaborator Author

Cached run:

build: 4m15s
publish: 2m46s
check: 3m3s

I'm going to call this a success

@teunbrand teunbrand marked this pull request as ready for review January 22, 2026 11:00
@teunbrand teunbrand merged commit 6aa4444 into posit-dev:main Jan 22, 2026
6 checks passed
@teunbrand teunbrand deleted the caching_experiment branch January 22, 2026 11:01
@thomasp85
Copy link
Collaborator

Fantastic!

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