Skip to content

GH Actions: lower compiler cache size to 0.5 GB#4452

Merged
Holzhaus merged 1 commit intomixxxdj:mainfrom
Be-ing:cache_size
Oct 19, 2021
Merged

GH Actions: lower compiler cache size to 0.5 GB#4452
Holzhaus merged 1 commit intomixxxdj:mainfrom
Be-ing:cache_size

Conversation

@Be-ing
Copy link
Copy Markdown
Contributor

@Be-ing Be-ing commented Oct 17, 2021

Hopefully this will result in less frequent cache misses.

@Be-ing Be-ing marked this pull request as draft October 17, 2021 13:43
@Be-ing
Copy link
Copy Markdown
Contributor Author

Be-ing commented Oct 17, 2021

2 GB * 3 operating systems * 2 (Qt5 + Qt6) > 5 GB

@Be-ing
Copy link
Copy Markdown
Contributor Author

Be-ing commented Oct 17, 2021

Here is a run with all cache misses. The sizes of the compiler caches uploaded to GitHub Actions were:

  • Ubuntu: 376 MB
  • macOS: 211 MB
  • Windows: 263 MB

So I think 512 MB is sufficient.

@Be-ing Be-ing marked this pull request as ready for review October 17, 2021 14:21
Comment thread .github/workflows/build.yml Outdated
- name: "[Ubuntu/macOS] Set compiler cache size limit"
# Set size to 2 GiB
run: ${{ matrix.compiler_cache }} -M 2097152
# Set size to 512 MiB
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add a comment why this particular value has been chosen, i.e. the calculation from the PR description? Prefer to add important information directly to the code instead of only mentioning it in the PR.

Hopefully this will result in less frequent cache misses.
@Holzhaus Holzhaus merged commit b077d80 into mixxxdj:main Oct 19, 2021
@Be-ing Be-ing deleted the cache_size branch October 19, 2021 17:39
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.

3 participants