Skip to content

Use setup-gradle GitHub action#1092

Merged
breedx-splk merged 1 commit into
open-telemetry:mainfrom
embrace-io:fractalwrench/ci-gradle-action
Jul 25, 2025
Merged

Use setup-gradle GitHub action#1092
breedx-splk merged 1 commit into
open-telemetry:mainfrom
embrace-io:fractalwrench/ci-gradle-action

Conversation

@fractalwrench
Copy link
Copy Markdown
Member

@fractalwrench fractalwrench commented Jul 24, 2025

Goal

The setup-gradle Github Action offers better build performance on CI. It caches build state such as gradle JARs and output from build tasks between jobs which therefore avoids starting the build from scratch each time. I didn't spot any alternative solution that was doing this already.

If this approach is accepted I can raise a separate changeset to do this for the other CI workflows.

Testing

Relying on the CI job. I don't have permissions to retry the job to test the cache, which may be worth doing to get an idea of the speed improvements.

@fractalwrench fractalwrench requested a review from a team as a code owner July 24, 2025 10:43
@fractalwrench fractalwrench changed the title Use setup-gradle action Use setup-gradle GitHub action Jul 24, 2025
Copy link
Copy Markdown
Contributor

@bidetofevil bidetofevil left a comment

Choose a reason for hiding this comment

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

LGTM. Seems faster on the initial run

Copy link
Copy Markdown
Contributor

@LikeTheSalad LikeTheSalad left a comment

Choose a reason for hiding this comment

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

Thank you!

Copy link
Copy Markdown
Contributor

@breedx-splk breedx-splk left a comment

Choose a reason for hiding this comment

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

I'm excited about this -- we use it in the instrumentation builds as well. 🤞🏻

@breedx-splk breedx-splk merged commit 5c9fd3d into open-telemetry:main Jul 25, 2025
6 checks passed
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