Skip to content

Conversation

@zeitlinger
Copy link
Member

@zeitlinger zeitlinger commented Nov 24, 2025

@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Nov 24, 2025
@zeitlinger
Copy link
Member Author

@jack-berg it turns out that spring native needs the reflect-config.json added in this PR. I would propose we add this to the SDK directly, because everyone using declarative config with graal VM is going to have this issue.

@zeitlinger
Copy link
Member Author

@jack-berg it turns out that spring native needs the reflect-config.json added in this PR. I would propose we add this to the SDK directly, because everyone using declarative config with graal VM is going to have this issue.

SDK already has other runtime hints so I created open-telemetry/opentelemetry-java#7866

Tested that it works with

Index: smoke-tests-otel-starter/spring-boot-3.2/build.gradle.kts
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/smoke-tests-otel-starter/spring-boot-3.2/build.gradle.kts b/smoke-tests-otel-starter/spring-boot-3.2/build.gradle.kts
--- a/smoke-tests-otel-starter/spring-boot-3.2/build.gradle.kts	(revision 78636198f5ce1b4d8d6aeed8fd91aba228bfb068)
+++ b/smoke-tests-otel-starter/spring-boot-3.2/build.gradle.kts	(date 1764007502573)
@@ -31,6 +31,13 @@
   }
 }
 
+// todo only for local testing, remove before merging
+configurations.configureEach {
+  resolutionStrategy {
+    force("io.opentelemetry:opentelemetry-sdk-extension-incubator:1.57.0-alpha-SNAPSHOT")
+  }
+}
+
 springBoot {
   mainClass = "io.opentelemetry.spring.smoketest.OtelSpringStarterSmokeTestApplication"
 }
Index: mise.toml
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/mise.toml b/mise.toml
--- a/mise.toml	(revision 78636198f5ce1b4d8d6aeed8fd91aba228bfb068)
+++ b/mise.toml	(date 1764006988344)
@@ -1,4 +1,5 @@
 [tools]
+java = "graalvm-community-25.0.1"
 lychee = "0.21.0"
 
 [settings]

@zeitlinger zeitlinger self-assigned this Nov 24, 2025
@zeitlinger zeitlinger changed the title Fix spring starter native declarative config Add test for spring starter native declarative config Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test native This label can be applied to PRs to trigger them to run native tests

Projects

Status: Blocked

Development

Successfully merging this pull request may close these issues.

1 participant