Skip to content

Improve CI job time#945

Merged
dakom merged 16 commits into
mainfrom
935-see-if-we-can-improve-ci-caching
Sep 19, 2025
Merged

Improve CI job time#945
dakom merged 16 commits into
mainfrom
935-see-if-we-can-improve-ci-caching

Conversation

@ueco-jb
Copy link
Copy Markdown
Member

@ueco-jb ueco-jb commented Sep 18, 2025

Despite all tries and executions, I didn't manage to find any real improvements.
We can merge it if you agree with the changes (at least the actions/checkout got upgraded to v4), but overall CI time stayed similar:

  • 3:40 to build tests fresh
  • 1:40 to build tests with cache (subsequent runs)
  • 7:14 to run tests afterwards (this is consistent)

@ueco-jb ueco-jb self-assigned this Sep 18, 2025
@ueco-jb ueco-jb linked an issue Sep 18, 2025 that may be closed by this pull request
@ismellike
Copy link
Copy Markdown
Collaborator

Can exclude wavs-benchmark-common and wavs-dev-tools

@ueco-jb ueco-jb force-pushed the 935-see-if-we-can-improve-ci-caching branch from ab73e2f to d600a99 Compare September 18, 2025 20:00
@ueco-jb ueco-jb force-pushed the 935-see-if-we-can-improve-ci-caching branch from d600a99 to 5e7fdb2 Compare September 18, 2025 21:25
@ueco-jb ueco-jb marked this pull request as ready for review September 18, 2025 22:24
Copy link
Copy Markdown
Collaborator

@dakom dakom left a comment

Choose a reason for hiding this comment

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

LGTM!

@dakom dakom enabled auto-merge September 19, 2025 10:22
@dakom dakom disabled auto-merge September 19, 2025 10:26
@dakom dakom enabled auto-merge September 19, 2025 10:26
@dakom dakom disabled auto-merge September 19, 2025 10:28
@dakom dakom merged commit e40ba16 into main Sep 19, 2025
2 checks passed
@dakom dakom deleted the 935-see-if-we-can-improve-ci-caching branch September 19, 2025 10:28
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.

See if we can improve CI caching

3 participants