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

Speedup ogc tests #38714

Merged
merged 2 commits into from Sep 13, 2020
Merged

Speedup ogc tests #38714

merged 2 commits into from Sep 13, 2020

Conversation

m-kuhn
Copy link
Member

@m-kuhn m-kuhn commented Sep 12, 2020

Description

Two improvements to ogc testing ci

  • Disable ccache, it has a 0% hit rate at the moment and I'm not aware how to improve things with github workflows
  • Disable building of tests. We do that in other workflows and it just adds to the build time here.

1h straight now vs. 1h 22 minutes before = 27% faster

Github cache is immutable.
It's useful for caching static dependencies like pip, but not for dynamic caches like ccache.
It has a 0% hit rate and only adds overhead at the moment.
To be revisited when github caching improves.
@github-actions github-actions bot added this to the 3.16.0 milestone Sep 12, 2020
@pblottiere
Copy link
Member

1h straight now vs. 1h 22 minutes before = 27% faster

Nice, thanks Matthias 👍

@m-kuhn m-kuhn merged commit 2a3fdf1 into qgis:master Sep 13, 2020
@m-kuhn m-kuhn deleted the speedup_ogc_tests branch September 13, 2020 08:23
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.

None yet

2 participants