MINOR: Upgrade gradle to 9.7.0 - #23142
Conversation
Update Gradle from 9.6.1 to 9.7.0, including the wrapper distribution checksum and Kafka's wrapper JAR checksum references. Followed the procedure in gradle/wrapper/README.md: - `gradle/dependencies.gradle`: gradle version - `wrapper.gradle`: wrapperChecksum - `gradle/wrapper/gradle-wrapper.properties` and `gradlew` regenerated with `./gradlew wrapper --gradle-version 9.7.0 --distribution-type bin --gradle-distribution-sha256-sum <bin-zip checksum>` The regenerated `gradle-wrapper.properties` also adds `retries=0` and `retryBackOffMs=500`. These are the `WrapperConfiguration` defaults that 9.7.0 now writes out explicitly, so behaviour is unchanged. Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
|
Thank you for noticing my work @serhiy-bzhezytskyy ! At the moment I have three pending Gradle related PR's but none of them depends on a specific Gradle version. All-in-all: go for it. Note: I could provide some non-binding review in a few days (can't do that via mobile phone app) but hopefully some maintainer will pick this up sooner. |
|
Thanks @dejan2609!
The exit code then comes from The distribution itself is fine — I fetched Could someone re-run the failed job when convenient? |
@serhiy-bzhezytskyy (Not sure is there an option to rerun only failed jobs; maybe maintainers could do that) |
|
@serhiy-bzhezytskyy Thanks for the update and following up! The current failures look like flaky tests. I agree with @dejan2609 that you can sync with trunk and push a merge commit to re-trigger the CI. That's usually how I handle these CI failures too. |
|
Two new entries in Related links:
I did some testing:
@serhiy-bzhezytskyy 💡 Hint: bump Gradle version to 9.7.1 that will be released in a next few days: https://github.com/gradle/gradle/milestone/352?closed=1 Overall: LGTM (non-binding) ✅ |
Update Gradle from 9.6.1 to 9.7.0, including the wrapper distribution
checksum and Kafka's wrapper JAR checksum references.
Followed the procedure in
gradle/wrapper/README.md:gradle/dependencies.gradle— gradle versionwrapper.gradle—wrapperChecksumgradle/wrapper/gradle-wrapper.propertiesandgradlewregenerated with./gradlew wrapper --gradle-version 9.7.0 --distribution-type bin --gradle-distribution-sha256-sum <checksum>Checksums are the published ones from https://gradle.org/release-checksums/:
binary distribution
84fbba45…873ae, wrapper JAR7a9ce74c…62c5d.The regenerated
gradle-wrapper.propertiesalso picks upretries=0andretryBackOffMs=500. Those are theWrapperConfigurationdefaults which9.7.0 now writes out explicitly, so behaviour is unchanged.
Verified locally with Gradle 9.7.0
./gradlew --versionreports Gradle 9.7.0Compile and artifacts
./gradlew compileJava compileTestJava./gradlew jar./gradlew srcJar./gradlew testJar./gradlew docsJarGenerated messages
./gradlew processMessages processTestMessagesJavadoc and scaladoc
./gradlew javadoc javadocJar./gradlew scaladoc scaladocJar./gradlew aggregatedJavadoc --no-parallelCode quality
./gradlew clients:checkstyleMain clients:checkstyleTest clients:spotlessCheck./gradlew clients:spotbugsMain -x testTests and publishing
./gradlew clients:test --tests RequestResponseTest./gradlew -PskipSigning=true publishToMavenLocalThe build reports "Deprecated Gradle features were used in this build,
making it incompatible with Gradle 10." I ran the same tasks on 9.6.1 and
the set of reported deprecations is identical, so this is pre-existing and
not introduced here.
I did not run
releaseTarGz, the full test suite, or the system testslocally; leaving those to CI.
Following the
MINOR:form used for the 9.6.1 bump in #23015, so no JIRA.@dejan2609 you have been driving the Gradle and build-tooling updates
lately, so flagging this in case it overlaps with anything you have in
flight. @Parkerhiphop this follows your #23015 directly.
Reviewers: Dejan Stojadinović dejan2609@gmail.com, Parker Chang parkerhiphop027@gmail.com