ci: default Kotlin gradle-args to --parallel --build-cache [NOJIRA]#318
ci: default Kotlin gradle-args to --parallel --build-cache [NOJIRA]#318tobias0106 wants to merge 1 commit into
Conversation
…ows [NOJIRA] Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Be careful making this default across all projects - this can break stuff if tests are not supporting parallel runs - perhaps introduce a parameter instead ? |
morten-andersen
left a comment
There was a problem hiding this comment.
I like it - but a scary change - we risk blocking a lot of kotlin services - and quite a few people are out for vacation in many teams
Fair point. Also a risky PR. |
Would just be cool to streamline things across all Kotlin services, but I agree it's not worth the risk. Appreciate the feedback! :praise |
Why
service-wallet#4478 sped up its Sonar step by overriding
gradle-argsin the caller:--parallel --build-cacheinstead of the shared default--no-daemon --parallel. Dropping--no-daemonlets the test/sonar/coverage steps share one warm Gradle daemon, and--build-cachelets later invocations reuse outputs.This promotes that optimization to the shared Kotlin workflows so every Kotlin service gets it as the default, instead of each repo overriding per-caller.
What
Change the
gradle-argsinput default from--no-daemon --parallelto--parallel --build-cachein all reusable Kotlin/Gradle workflows:pull-request-kotlin.ymlsonar-cloud.ymlcode-coverage-kotlin.ymlcomponent-test-kotlin.ymldeploy-kotlin.ymldeploy-kotlin-v2.ymlInput-only change — callers that already pass an explicit
gradle-argsare unaffected.Risk / tradeoffs
large/blacksmith runners this is the intended setup.gradle-args: "--no-daemon --parallel"explicitly.Draft — open for review before un-drafting.
🤖 Generated with Claude Code