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

feat: add remote build caching #19390

Merged
merged 2 commits into from
Nov 14, 2022
Merged

feat: add remote build caching #19390

merged 2 commits into from
Nov 14, 2022

Conversation

c-p-b
Copy link
Contributor

@c-p-b c-p-b commented Nov 14, 2022

What

Adds remote caching for gradle builds

How

Uses https://github.com/burrunan/gradle-s3-build-cache

@c-p-b c-p-b temporarily deployed to more-secrets November 14, 2022 19:24 Inactive
@c-p-b c-p-b marked this pull request as ready for review November 14, 2022 21:03
@c-p-b c-p-b requested a review from a team November 14, 2022 21:04
@c-p-b
Copy link
Contributor Author

c-p-b commented Nov 14, 2022

Octavia CLI: 10 minutes faster with caching
Platform: 4 minutes faster with caching
Connectors Base: 2 minutes faster with caching

Before: https://github.com/airbytehq/airbyte/actions/runs/3464245652/jobs/5785566709
After: https://github.com/airbytehq/airbyte/actions/runs/3464506956/jobs/5786145592

@supertopher
Copy link
Contributor

those are some pretty sick gains. nice find conor

@c-p-b c-p-b merged commit 9bf5dac into master Nov 14, 2022
@c-p-b c-p-b deleted the remote-cache branch November 14, 2022 21:10
ext.isCiServer = System.getenv().containsKey("CI")

buildCache {
local {
Copy link
Contributor

Choose a reason for hiding this comment

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

@cpdeethree can we enable this? If we are afraid of this edge case, I think education is a better path here.

akashkulk pushed a commit that referenced this pull request Dec 2, 2022
* feat: add remote build caching

* ci: retrigger actions
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