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

Improve Cloud Build build times #176

Merged
merged 1 commit into from
Jan 28, 2021
Merged

Improve Cloud Build build times #176

merged 1 commit into from
Jan 28, 2021

Conversation

markmandel
Copy link
Contributor

@markmandel markmandel commented Jan 28, 2021

This implements several performance improvements:

  • Persist CARGO_HOME between CI steps through the CARGO_HOME env
    variable.
  • Removes the redundant cargo check step
  • Increases size of the build VM to a N1_HIGHCPU_8

This moves the build time down from ~25m to ~5m raised_hands

Closes #173

@markmandel markmandel added area/tests Unit tests, integration tests, anything to make sure things don't break kind/cleanup Refactoring code, fixing up documentation, etc labels Jan 28, 2021
@google-cla google-cla bot added the cla: yes label Jan 28, 2021
@markmandel markmandel changed the title Persist CARGO_HOME between CI steps Improve Cloud Build build times Jan 28, 2021
This implements several performance improvments:
* Persist CARGO_HOME between CI steps through the CARGO_HOME env
  variable.
* Removes the redundant `cargo check` step
* Increases size of the build VM to a N1_HIGHCPU_8

This moves the build time down from ~25m to ~5m 🙌

Closes #173
@markmandel markmandel marked this pull request as ready for review January 28, 2021 03:17
@markmandel markmandel requested a review from iffyio January 28, 2021 03:17
Copy link
Collaborator

@iffyio iffyio left a comment

Choose a reason for hiding this comment

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

🎉

@markmandel markmandel merged commit 7747afa into main Jan 28, 2021
@markmandel markmandel deleted the ci/cargo-home branch January 28, 2021 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tests Unit tests, integration tests, anything to make sure things don't break cla: yes kind/cleanup Refactoring code, fixing up documentation, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Reuse downloaded / compiled libraries between steps
2 participants