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

CI: Use caches to pass data between jobs #1608

Merged
merged 3 commits into from
May 5, 2019
Merged

Conversation

effigies
Copy link
Member

@effigies effigies commented May 5, 2019

This one is fiddling with Circle caches, which we persist anyway, to pass data between jobs. Saving to cache seems to be slightly faster than persisting to the workspace, so this:

  1. De-duplicates saving cached/workspace data, choosing the cheaper option.
  2. Eliminates the separate update_cache job, which persists, reloads and then caches.

The savings of the save/load steps seems to be within the variance of the whole job, so I'm not sure that we'll see much of a speed up (I expect the effect is there, but ~2 minutes in 30-70 minute jobs is not much to brag of. That said, this does eliminate a 7 minute job.

I also experimented with reducing the compression level to 3 and only using 2 threads, and that appears to shave 3-5 minutes off of the save step while not significantly adding to the load step. We can fiddle with this more if there's interest in being systematic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants