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

Speed up CI #75

Merged
merged 26 commits into from
Jan 15, 2023
Merged

Speed up CI #75

merged 26 commits into from
Jan 15, 2023

Conversation

kaspth
Copy link
Contributor

@kaspth kaspth commented Jan 15, 2023

Looks like we have several steps in the starter repo minitest CI job that could be sped up. So this tries some things.

@kaspth
Copy link
Contributor Author

kaspth commented Jan 15, 2023

I've been trying stuff for 1.5 hours and have made absolutely no progress, and I've scoured CircleCI docs for help on cache generation and storage. All we're doing is changing the working_directory to a subdirectory for a specific CI build and CircleCI cannot figure anything out. Great stuff, really cool.

@kaspth kaspth force-pushed the speed-up-ci branch 2 times, most recently from b0293ba to 1574dce Compare January 15, 2023 19:56
@kaspth kaspth force-pushed the speed-up-ci branch 4 times, most recently from f2ff482 to 4405576 Compare January 15, 2023 21:42
@kaspth
Copy link
Contributor Author

kaspth commented Jan 15, 2023

Ok, so looks like I somehow got bundle install caching partly working and saving us a full minute. We went from ~1:20 min to ~20 seconds (I don't know why it's 20 seconds and not 0).

I also cached the Firefox install by keeping us to a version that ships with the image, which saves 20-30 seconds.

@kaspth kaspth merged commit f6f6975 into main Jan 15, 2023
@kaspth kaspth deleted the speed-up-ci branch January 15, 2023 21:46
@kaspth
Copy link
Contributor Author

kaspth commented Jan 15, 2023

We can hopefully port all these things to bullet_train-core.

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

1 participant