adityamparikh opened a new pull request, #217:
URL: https://github.com/apache/solr-mcp/pull/217

   ## Summary
   
   `opentelemetry-instrumentation-bom` and `opentelemetry-spring-boot-starter` 
were hardcoded string literals in `build.gradle.kts`, invisible to 
catalog-driven dependency tooling. #82 bumped `gradle/libs.versions.toml`'s 
`opentelemetry-instrumentation-bom` to `2.26.1` (a CVE fix) but that bump never 
took effect, because `build.gradle.kts` still pinned the literal `2.11.0` — the 
catalog entry was silently orphaned.
   
   This wires both dependencies through the version catalog 
(`libs.opentelemetry.instrumentation.bom`, 
`libs.opentelemetry.spring.boot.starter`) so future catalog bumps actually 
apply.
   
   The catalog version stays at `2.11.0` — this is a build-wiring fix only, 
**no functional version change**. `2.26.1` needs 
`io.opentelemetry.common.ComponentLoader`, which Spring Boot 3.5.x's managed 
OTel SDK doesn't ship yet, and breaks native AOT (documented in 
`dev-docs/graalvm-native-image.md`, "Why not just bump OTel?"). I added a 
comment on the catalog entry pointing at that doc so the pin doesn't look like 
an oversight again.
   
   ## Test plan
   
   - [x] `./gradlew assemble` — builds clean
   - [x] `./gradlew build -x rat` (JDK 25 on PATH) — full build, all tests, 
`spotlessApply` all pass
   - [x] Confirmed `:rat` failure is pre-existing/local-only (untracked 
`.playwright-mcp/*.yml` files), unrelated to this change


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to