-
Notifications
You must be signed in to change notification settings - Fork 19
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
Optimize ci #630
Optimize ci #630
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That sounds great!
Is it a free solution?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice, and I didn't know about warp builds until this PR. Looks great to add
https://docs.warpbuild.com/runners |
Terraform Feature Environment Destroy (dev-630)Terraform Initialization ⚙️
|
@volovyks it's not free but quite cheap. I'll give a try on their mac m1 machines, maybe they work unlike github's |
We may need to build M1 Docker images, I'm not sure if it is required to run tests on M1 instances. |
Don't think it would change anything though, since what was broken wasn't that we didn't have M1 docker builds but that colima (the docker virtualization tooling) was flaky on macos and we never bothered to get it working again. |
Previous cache has a size limit of 10G and it is not sufficient, result in fresh build everytime. This PR uses a unlimited cache to reduce build time. In the best case where dependency crate is not changed and only our code changes, it reduce 20 minutes of build time of unit (28min -> 6min), fastauth integration (33min -> 13min) and chain signature integration tests (35min -> 16min).