Skip to content

Commit 8199b4c

Browse files
committed
chore: Additional lazy API changes
1 parent 24adc9a commit 8199b4c

File tree

13 files changed

+36
-31
lines changed

13 files changed

+36
-31
lines changed

dd-java-agent/ddprof-lib/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ apply from: "$rootDir/gradle/java.gradle"
99
dependencies {
1010
// This module provides the ddprof library as an api dependency
1111
// so that other modules can easily depend on it.
12-
implementation project.hasProperty('ddprof.jar') ? files(project.getProperty('ddprof.jar')) : libs.ddprof
12+
implementation project.hasProperty('ddprof.jar') ? files(project.findProperty('ddprof.jar')) : libs.ddprof
1313
api project(':internal-api')
1414
api project(':dd-trace-api')
1515
}

dd-java-agent/instrumentation/build.gradle

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,13 +122,12 @@ dependencies {
122122

123123
if (project.gradle.startParameter.taskNames.any { it.endsWith("generateMuzzleReport") }) {
124124
apply plugin: 'muzzle'
125-
task("muzzleInstrumentationReport") {
126-
dependsOn(project.getAllTasks(true).values().flatten().findAll { it.name.endsWith("generateMuzzleReport") })
125+
tasks.register('muzzleInstrumentationReport') {
126+
dependsOn(subprojects.collect { p -> p.tasks.matching { it.name == "generateMuzzleReport" } })
127127
finalizedBy(tasks.named('mergeMuzzleReports'))
128128
}
129129
}
130130

131-
132131
tasks.named('shadowJar', ShadowJar) {
133132
duplicatesStrategy = DuplicatesStrategy.FAIL
134133
dependencies {

dd-java-agent/instrumentation/grizzly/grizzly-client-1.9/build.gradle

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,15 @@ apply from: "$rootDir/gradle/java.gradle"
2323
addTestSuiteForDir('latestDepTest', 'test')
2424
addTestSuiteForDir('muleLatestDepTest', 'test')
2525

26+
configurations.named("muleLatestDepTestRuntimeClasspath") {
27+
exclude(group: 'org.glassfish.grizzly')
28+
}
29+
2630
dependencies {
2731
compileOnly group: 'org.glassfish.grizzly', name: 'grizzly-http-client', version: '1.9'
2832
// for some reason, the tests don't *load* until 1.12, but muzzles works as far back as 1.9
2933
testImplementation group: 'org.glassfish.grizzly', name: 'grizzly-http-client', version: '1.12'
3034

3135
latestDepTestImplementation group: 'org.glassfish.grizzly', name: 'grizzly-http-client', version: '+'
3236
muleLatestDepTestImplementation group: 'org.mule.glassfish.grizzly', name: 'grizzly-http-client', version: '+'
33-
configurations.getByName("muleLatestDepTestRuntimeClasspath").exclude(group: 'org.glassfish.grizzly')
3437
}

dd-java-agent/instrumentation/grizzly/grizzly-http-2.3.20/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,6 @@ dependencies {
3434
latestDepTestImplementation group: 'org.glassfish.jersey.inject', name: 'jersey-hk2', version: '2.+'
3535
}
3636

37-
tasks.withType(Test) {
37+
tasks.withType(Test).configureEach {
3838
jvmArgs += ['-Ddd.integration.grizzly-filterchain.enabled=true']
3939
}

dd-java-agent/instrumentation/jersey/build.gradle

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,15 @@ dependencies {
7777
jersey3JettyTestRuntimeOnly project(':dd-java-agent:instrumentation:jakarta-rs-annotations-3')
7878
}
7979

80-
configurations.getByName('jersey3JettyTestRuntimeClasspath').resolutionStrategy {
81-
force libs.slf4j
80+
configurations.named('jersey3JettyTestRuntimeClasspath') {
81+
resolutionStrategy {
82+
force libs.slf4j
83+
}
8284
}
83-
configurations.getByName('jersey2JettyTestRuntimeClasspath').resolutionStrategy {
84-
// override version in testImplementation
85-
force "org.glassfish.jersey.core:jersey-common:${jersey2Version}"
86-
force "org.eclipse.jetty:jetty-server:9.1.1.v20140108"
87-
85+
configurations.named('jersey2JettyTestRuntimeClasspath') {
86+
resolutionStrategy {
87+
// override version in testImplementation
88+
force "org.glassfish.jersey.core:jersey-common:${jersey2Version}"
89+
force "org.eclipse.jetty:jetty-server:9.1.1.v20140108"
90+
}
8891
}

dd-java-agent/instrumentation/servlet/jakarta-servlet-5.0/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ configurations {
3030
javaxClassesToRelocate
3131
}
3232

33-
task relocatedJavaxJar(type: ShadowJar) {
33+
tasks.register('relocatedJavaxJar', ShadowJar) {
3434
relocate 'javax.servlet', 'jakarta.servlet'
3535
relocate 'datadog.trace.instrumentation.servlet3', 'datadog.trace.instrumentation.servlet5'
3636
relocate 'datadog.trace.instrumentation.servlet', 'datadog.trace.instrumentation.servlet5'

dd-smoke-tests/appsec/spring-tomcat7/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ tasks.withType(Test).configureEach {
2626
jvmArgs "-Ddatadog.smoketest.appsec.springtomcat7.shadowJar.path=${tasks.shadowJar.archiveFile.get()}"
2727
}
2828

29-
task testRuntimeActivation(type: Test) {
29+
tasks.register('testRuntimeActivation', Test) {
3030
jvmArgs '-Dsmoke_test.appsec.enabled=inactive',
3131
"-Ddatadog.smoketest.appsec.springtomcat7.shadowJar.path=${tasks.shadowJar.archiveFile.get()}"
3232
}

dd-smoke-tests/appsec/springboot-grpc/build.gradle

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import com.github.jengelman.gradle.plugins.shadow.tasks.ShadowJar
2+
13
plugins {
24
id 'com.gradleup.shadow'
35
}
@@ -26,7 +28,9 @@ tasks.withType(Test).configureEach {
2628
evaluationDependsOn ':dd-smoke-tests:springboot-grpc'
2729
dependsOn ':dd-smoke-tests:springboot-grpc:shadowJar'
2830

29-
Task shadowJarTask = project(':dd-smoke-tests:springboot-grpc').tasks['shadowJar']
31+
def shadowJarTask = project(':dd-smoke-tests:springboot-grpc').tasks.named('shadowJar', ShadowJar)
3032

31-
jvmArgs "-Ddatadog.smoketest.appsec.springboot-grpc.shadowJar.path=${shadowJarTask.archiveFile.get()}"
33+
jvmArgumentProviders.add({
34+
["-Ddatadog.smoketest.appsec.springboot-grpc.shadowJar.path=${shadowJarTask.get().archiveFile.get()}"]
35+
})
3236
}

dd-smoke-tests/appsec/springboot/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ tasks.withType(Test).configureEach {
3434
jvmArgs "-Ddatadog.smoketest.appsec.springboot.shadowJar.path=${tasks.shadowJar.archiveFile.get()}"
3535
}
3636

37-
task testRuntimeActivation(type: Test) {
37+
tasks.register('testRuntimeActivation', Test) {
3838
jvmArgs '-Dsmoke_test.appsec.enabled=inactive',
3939
"-Ddatadog.smoketest.appsec.springboot.shadowJar.path=${tasks.shadowJar.archiveFile.get()}"
4040
}

dd-smoke-tests/datastreams/kafkaschemaregistry/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ tasks.withType(Test).configureEach {
3131
jvmArgs "-Ddatadog.smoketest.datastreams.kafkaschemaregistry.shadowJar.path=${tasks.shadowJar.archiveFile.get()}"
3232
}
3333

34-
task testRuntimeActivation(type: Test) {
34+
tasks.register('testRuntimeActivation', Test) {
3535
jvmArgs "-Ddatadog.smoketest.datastreams.kafkaschemaregistry.shadowJar.path=${tasks.shadowJar.archiveFile.get()}"
3636
}
3737

0 commit comments

Comments
 (0)