Skip to content

Conversation

@nmdefries
Copy link
Contributor

Description

Switch survey CI to use daily caching.

Changelog

  • Use full date ("YYYYMMDD") in cache name instead of month.

@nmdefries nmdefries marked this pull request as ready for review July 21, 2021 15:22
@nmdefries nmdefries requested a review from chinandrew July 21, 2021 15:29
@chinandrew
Copy link
Contributor

This means one run a day will take a while right?

@nmdefries
Copy link
Contributor Author

nmdefries commented Jul 21, 2021

No, it should only take a few minutes (the one here took 4 min total starting from the July cache). Each new daily run will load the cache from the previous day, which already has most of the required packages, so we only have to run incremental updates, we don't have to reinstall everything. And then the old packages + updates get saved to the new cache key.

Copy link
Contributor

@chinandrew chinandrew left a comment

Choose a reason for hiding this comment

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

:shipit::shipit::shipit:

@krivard krivard merged commit f8b46ea into main Jul 21, 2021
@krivard krivard deleted the survey-ci-daily-caching branch July 21, 2021 21:38
@krivard krivard mentioned this pull request Jul 26, 2021
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