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

[WIP] Try to speed up the CI for testing by building and caching in ghcr #972

Open
wants to merge 41 commits into
base: master
Choose a base branch
from

Conversation

suricactus
Copy link
Collaborator

No description provided.

@duke-nyuki
Copy link
Collaborator

Task linked: QF-4434 Try to optimize CI test builds

.github/workflows/test.yml Outdated Show resolved Hide resolved
@anajoy
Copy link
Collaborator

anajoy commented Sep 26, 2024

Build with ghcr.io caching of qfieldcloud images did run (test.yml):
7 m 07 s (app: 1 m 31 s, nginx: 19 s, qgis: 5 m 17 s)
2 m 49 s (app: 2 m 15 s, nginx: 21 s, qgis: 0 m 13 s)
3 m 30 s (app: 2 m 36 s, nginx: 41 s, qgis: 0 m 13 s)
3 m 15 s (app: 2 m 23 s, nginx: 36 s, qgis: 0 m 16 s)
6 m 39 s (app: 1 m 21 s, nginx: 21 s, qgis: 5 m 57 s, reaching 10 GB of cached layers
It looks like, I have reached the limit of available github action caches of 10 GB. Deleting all caches for pr 972 to retest caching
8 m 59 s (app: 2 m 26 s, nginx: 24 s, qgis: 7 m 09 s, first ghcr.io cache created , size 2.22 GB
8 m 53 s (app: 2 m 31 s, nginx: 19 s, qgis: 6 m 03 s, cache not used ?, size 4.44 GB
2 m 54 s (app: 2 m 29 s, nginx: 18 s, qgis: 0 m 17 s, cache used !, size 4.44 GB

Build without ghcr.io caching of qfieldcloud images did run (test_baseline.yml):
6 m 40 s (app + nginx + qgis build_and_push without the push step)
6 m 10 s (app + nginx + qgis build_and_push without the push step)
6 m 10 s (app + nginx + qgis build_and_push without the push step)
6 m 07 s (app + nginx + qgis build_and_push without the push step)
7 m 23 s (app + nginx + qgis build_and_push without the push step)
6 m 45 s (app + nginx + qgis build_and_push without the push step)

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.

4 participants