Skip to content

Commit 05a6834

Browse files
authored
Merge branch 'master' into smola/telemetry-logs-tags
2 parents 1fd875f + c08d6d9 commit 05a6834

10 files changed

Lines changed: 32 additions & 19 deletions

File tree

.circleci/collect_results.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ function get_source_file () {
3535
fi
3636
done
3737
done < <(grep -rl "class $class" "$file_path")
38-
file_path="$common_root"
38+
file_path="/$common_root"
3939
fi
4040
}
4141

.github/workflows/update-docker-build-image.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
echo "::notice::Using Docker build image tag: ${TAG}"
5353
- name: Update the Docker build image in GitLab CI config
5454
run: |
55-
sed -i 's|JAVA_BUILD_IMAGE_VERSION:.*|JAVA_BUILD_IMAGE_VERSION:"${{ steps.define-tag.outputs.tag }}"|' .gitlab-ci.yml
55+
sed -i 's|BUILDER_IMAGE_VERSION_PREFIX:.*|BUILDER_IMAGE_VERSION_PREFIX:"${{ steps.define-tag.outputs.tag }}-"|' .gitlab-ci.yml
5656
- name: Commit and push changes
5757
env:
5858
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}

.gitlab-ci.yml

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ variables:
2727
GRADLE_VERSION: "8.5" # must match gradle-wrapper.properties
2828
MAVEN_REPOSITORY_PROXY: "http://artifactual.artifactual.all-clusters.local-dc.fabric.dog:8081/repository/maven-central/"
2929
GRADLE_PLUGIN_PROXY: "http://artifactual.artifactual.all-clusters.local-dc.fabric.dog:8081/repository/gradle-plugin-portal-proxy/"
30-
JAVA_BUILD_IMAGE_VERSION: "v25.05"
30+
BUILDER_IMAGE_VERSION_PREFIX: "v25.05-"
3131
REPO_NOTIFICATION_CHANNEL: "#apm-java-escalations"
3232
DEFAULT_TEST_JVMS: /^(8|11|17|21)$/
3333
PROFILE_TESTS:
@@ -97,7 +97,7 @@ default:
9797
- echo "NORMALIZED_NODE_TOTAL=${NORMALIZED_NODE_TOTAL}, NORMALIZED_NODE_INDEX=$NORMALIZED_NODE_INDEX"
9898

9999
.gradle_build: &gradle_build
100-
image: ghcr.io/datadog/dd-trace-java-docker-build:${JAVA_BUILD_IMAGE_VERSION}-base
100+
image: ghcr.io/datadog/dd-trace-java-docker-build:${BUILDER_IMAGE_VERSION_PREFIX}base
101101
stage: build
102102
variables:
103103
MAVEN_OPTS: "-Xms64M -Xmx512M"
@@ -228,7 +228,7 @@ spotless:
228228

229229
test_published_artifacts:
230230
extends: .gradle_build
231-
image: ghcr.io/datadog/dd-trace-java-docker-build:${JAVA_BUILD_IMAGE_VERSION}-7 # Needs Java7 for some tests
231+
image: ghcr.io/datadog/dd-trace-java-docker-build:${BUILDER_IMAGE_VERSION_PREFIX}7 # Needs Java7 for some tests
232232
stage: tests
233233
needs: [ build ]
234234
variables:
@@ -366,7 +366,7 @@ muzzle-dep-report:
366366

367367
.test_job:
368368
extends: .gradle_build
369-
image: ghcr.io/datadog/dd-trace-java-docker-build:$testJvm
369+
image: ghcr.io/datadog/dd-trace-java-docker-build:${BUILDER_IMAGE_VERSION_PREFIX}$testJvm
370370
tags: [ "docker-in-docker:amd64" ] # use docker-in-docker runner for testcontainers
371371
needs: [ build_tests ]
372372
stage: tests
@@ -387,7 +387,7 @@ muzzle-dep-report:
387387
when: on_success
388388
- if: $NON_DEFAULT_JVMS == "true"
389389
when: on_success
390-
- if: $CI_PIPELINE_SOURCE == "schedule"
390+
- if: $CI_COMMIT_BRANCH == "master"
391391
when: on_success
392392
script:
393393
- >
@@ -557,9 +557,7 @@ test_smoke:
557557

558558
test_ssi_smoke:
559559
extends: .test_job
560-
rules:
561-
- if: $CI_PIPELINE_SOURCE == "schedule"
562-
when: on_success
560+
rules: *master_only
563561
variables:
564562
GRADLE_TARGET: "stageMainDist :smokeTest"
565563
CACHE_TYPE: "smoke"

BUILDING.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ Requirements to build the full project:
6060

6161
### Install the required JDKs
6262

63-
Download and install Eclipse Temurin JDK versions 8, 11, 17 and 21, and GraalVM 17.
63+
Download and install JDK versions 8, 11, 17 and 21, and GraalVM 17 for your OS.
6464

6565
#### macOS
6666

@@ -179,17 +179,17 @@ scoop install git
179179
180180
#### macOS
181181

182-
Download and install Docker Desktop from the offical website:<br/>
182+
Download and install Docker Desktop from the official website:<br/>
183183
https://docs.docker.com/desktop/setup/install/mac-install/
184184

185185
#### Linux
186186

187-
Download and install Docker Desktop from the offical website:<br/>
187+
Download and install Docker Desktop from the official website:<br/>
188188
https://docs.docker.com/desktop/setup/install/linux/
189189

190190
#### Windows
191191

192-
Download and install Docker Desktop from the offical website:<br/>
192+
Download and install Docker Desktop from the official website:<br/>
193193
https://docs.docker.com/desktop/setup/install/windows-install/
194194

195195
<details>

dd-java-agent/agent-iast/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ java {
1515
toolchain {
1616
languageVersion.set(JavaLanguageVersion.of(11))
1717
}
18-
sourceCompatibility = 1.8
19-
targetCompatibility = 1.8
18+
sourceCompatibility = JavaVersion.VERSION_1_8
19+
targetCompatibility = JavaVersion.VERSION_1_8
2020
}
2121

2222
tasks.withType(AbstractCompile).configureEach {

dd-java-agent/agent-jmxfetch/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ plugins {
1111
apply from: "$rootDir/gradle/java.gradle"
1212

1313
dependencies {
14-
api('com.datadoghq:jmxfetch:0.49.7') {
14+
api('com.datadoghq:jmxfetch:0.49.8') {
1515
exclude group: 'org.slf4j', module: 'slf4j-api'
1616
exclude group: 'org.slf4j', module: 'slf4j-jdk14'
1717
exclude group: 'com.beust', module: 'jcommander'

dd-java-agent/benchmark-integration/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ apply from: "$rootDir/gradle/java.gradle"
2525

2626
description = 'Integration Level Agent benchmarks.'
2727

28-
sourceCompatibility = 1.8
29-
targetCompatibility = 1.8
28+
sourceCompatibility = JavaVersion.VERSION_1_8
29+
targetCompatibility = JavaVersion.VERSION_1_8
3030

3131
subprojects { sub ->
3232
sub.apply plugin: 'com.gradleup.shadow'

dd-java-agent/instrumentation/javax-xml/build.gradle

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,13 @@ repositories {
1414
}
1515
}
1616

17+
configurations.all {
18+
// shut up about broken xml-api pom relocation
19+
resolutionStrategy {
20+
force 'xml-apis:xml-apis:1.4.01'
21+
}
22+
}
23+
1724
dependencies {
1825
testRuntimeOnly project(':dd-java-agent:instrumentation:iast-instrumenter')
1926
testImplementation group: 'xalan', name: 'xalan', version: '2.7.0'

dd-java-agent/instrumentation/lettuce-4/src/test/groovy/Lettuce4ClientTestBase.groovy

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ abstract class Lettuce4ClientTestBase extends VersionedNamingTestBase {
8080

8181
def cleanup() {
8282
connection.close()
83+
redisClient.shutdown()
8384
redisServer.stop()
8485
}
8586
}

dd-java-agent/instrumentation/owasp-esapi-2/build.gradle

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,13 @@ muzzle {
77
}
88
}
99

10+
configurations.all {
11+
// shut up about broken xml-api pom relocation
12+
resolutionStrategy {
13+
force 'xml-apis:xml-apis:1.4.01'
14+
}
15+
}
16+
1017
apply from: "$rootDir/gradle/java.gradle"
1118
apply plugin: 'call-site-instrumentation'
1219

0 commit comments

Comments
 (0)