Skip to content

🍒 9955 - CSI plugin use too configuration classpath - #9965

Closed
amarziali wants to merge 1 commit into
release/v1.55.xfrom
andrea.marziali/backport-pr-9955
Closed

🍒 9955 - CSI plugin use too configuration classpath#9965
amarziali wants to merge 1 commit into
release/v1.55.xfrom
andrea.marziali/backport-pr-9955

Conversation

@amarziali

Copy link
Copy Markdown
Contributor

Backport #9955 to release/v1.55.x

The `generateCallSiteJava` task can fail when trying to resolve dependencies that have constraints, that can be set by the constraints DSL or by Dependency Locking (lockfile).
Indeed `CallSiteInstrumentationPlugin`'s `getProgramClasspath()` method is collecting classpaths from **all** `AbstractCompile` tasks, including `compileLatestDepForkedTestJava` and `compileLatestDepTestJava`, which can have conflicting version constraints
E.g. in javax-servlet-3.0
- testImplementation required strictly 8.2.0.v20160908
- latestDepTestImplementation wanted 9.+
- Dependency locking enforced 9.4.58.v20250814 for latestDepForkedTestCompileClasspath,latestDepForkedTestRuntimeClasspath,latestDepTestCompileClasspath,latestDepTestRuntimeClasspath
(cherry picked from commit 39eef9c)
@amarziali
amarziali requested a review from a team as a code ownerNovember 14, 2025 09:28
@amarzialiamarziali added type: bug fix Bug fix tag: no release notes Changes to exclude from release notes comp: tooling Build & Tooling labels Nov 14, 2025
@github-actions

Copy link
Copy Markdown
Contributor

Hi! 👋 Thanks for your pull request! 🎉

To help us review it, please make sure to:

  • Add at least one type, and one component or instrumentation label to the pull request

If you need help, please check our contributing guidelines.

@amarziali

Copy link
Copy Markdown
ContributorAuthor

mistake

@amarziali
amarziali deleted the andrea.marziali/backport-pr-9955 branch November 14, 2025 09:29
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: toolingBuild & Toolingtag: no release notesChanges to exclude from release notestype: bug fixBug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@amarziali@bric3