Skip to content

Commit b11249a

Browse files
T30: Fix edge CI Java version mismatch
Edge CI sets edgeDepsTest which compiles temporal-sdk targeting Java 21. Our module hardcoded Java 17, causing Gradle to reject the dependency at resolution time. Now uses 21 when edgeDepsTest is set, 17 otherwise. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 70c8594 commit b11249a

2 files changed

Lines changed: 21 additions & 5 deletions

File tree

TASK_QUEUE.json

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,16 @@
104104
"depends_on": [],
105105
"status": "completed",
106106
"notes": "Do in this PR, not a follow-up."
107+
},
108+
{
109+
"id": "T30",
110+
"title": "Fix CI: Edge build fails with Java version mismatch",
111+
"description": "Edge CI sets edgeDepsTest which compiles temporal-sdk at Java 21. Our module hardcodes Java 17, causing Gradle to reject the dependency. Fix: use 21 when edgeDepsTest is set, 17 otherwise.",
112+
"severity": "high",
113+
"category": "bugfix",
114+
"depends_on": [],
115+
"status": "completed",
116+
"notes": "Edge CI log showed: Dependency resolution is looking for a library compatible with JVM runtime version 17, but temporal-sdk is only compatible with JVM runtime version 21 or newer."
107117
}
108118
]
109119
}

temporal-spring-ai/build.gradle

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,20 +6,26 @@ ext {
66
springBootVersionForSpringAi = "$springBoot3Version"
77
}
88

9-
// Spring AI requires Java 17+, override the default Java 8 target from java.gradle
9+
// Spring AI requires Java 17+, override the default Java 8 target from java.gradle.
10+
// When edgeDepsTest is set, use 21 to match other modules (avoids Gradle JVM compatibility rejection).
11+
ext {
12+
springAiJavaVersion = project.hasProperty("edgeDepsTest") ? JavaVersion.VERSION_21 : JavaVersion.VERSION_17
13+
springAiRelease = project.hasProperty("edgeDepsTest") ? '21' : '17'
14+
}
15+
1016
java {
11-
sourceCompatibility = JavaVersion.VERSION_17
12-
targetCompatibility = JavaVersion.VERSION_17
17+
sourceCompatibility = springAiJavaVersion
18+
targetCompatibility = springAiJavaVersion
1319
}
1420

1521
compileJava {
1622
options.compilerArgs.removeAll(['--release', '8'])
17-
options.compilerArgs.addAll(['--release', '17'])
23+
options.compilerArgs.addAll(['--release', springAiRelease])
1824
}
1925

2026
compileTestJava {
2127
options.compilerArgs.removeAll(['--release', '8'])
22-
options.compilerArgs.addAll(['--release', '17'])
28+
options.compilerArgs.addAll(['--release', springAiRelease])
2329
}
2430

2531
dependencies {

0 commit comments

Comments
 (0)