Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CI ccache for library and restrict size #2866

Merged
merged 4 commits into from
Jul 18, 2024

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Jul 17, 2024

Fix CI library caching

#2865 broken library CI because:

  • ccache initialisation commands never get executed to set ccache directory
  • library ccache key must have toolchain in it

Fixed by removing the ccache commands from script and updating workflow env instead.
Just requires call to clear stats before building.

NB. This leads to a fair bit of duplication in all 3 workflow files. This is the simplest I can come up with for now.

Simplify workflows

Don't need the Configure environment step, just add values to env.

Use explicit ccache path

In testing was getting failures saving cache due to ccache not found.
Use explicit absolute path to ccache directory.

Restrict ccache size

The cache for each job will increase over time to hit 500MB.
Over 40 jobs that's 20GB and will cause cache thrashing.

Easy fix: ask ccache to evict anything older than 4 hours from ccache.
Do this before printing stats so we can see the resulting size.

@slaff slaff added this to the 5.2.0 milestone Jul 18, 2024
@slaff slaff merged commit 7c31a51 into SmingHub:develop Jul 18, 2024
46 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.

2 participants