Use env-entry to add tags per webapp deployment #8138

amarziali · 2024-12-31T14:21:43Z

What Does This Do

This PR concerns JEE web deployments only.

It adds the possibility to configure a set of tags to be added to each local root span per web deployment.
The tags are expressed as part of env-entry on the web.xml.

A tag can be added by declaring an env entry bindable on the namespace java:comp/env/datadog/tags/

An example:

<web-app> ... <env-entry> <env-entry-name>java:comp/env/datadog/tags/custom-tag</env-entry-name> <env-entry-type>java.lang.String</env-entry-type> <env-entry-value>some value</env-entry-value> </env-entry> ... </web-app>

This mechanism can also be used to configure a separate service name for each webapp since the tag service has a special meaning and TagInterceptor use its value to set the service name

Example:

<web-app> ... <env-entry> <env-entry-name>java:comp/env/datadog/tags/service</env-entry-name> <env-entry-type>java.lang.String</env-entry-type> <env-entry-value>>my-service</env-entry-value> </env-entry> ... </web-app>

That feature works today only with tomcat 9+ and wildfly 9+ .

Motivation

Additional Notes

Contributor Checklist

Format the title according the contribution guidelines
Assign the type: and (comp: or inst:) labels in addition to any usefull labels
Don't use close, fix or any linking keywords when referencing an issue.
Use solves instead, and assign the PR milestone to the issue
Update the public documentation in case of new configuration flag or behavior

Jira ticket: [PROJ-IDENT]

pr-commenter · 2024-12-31T15:01:27Z

Benchmarks

Startup

Load

Parameters

	Baseline	Candidate
baseline_or_candidate	baseline	candidate
end_time	2025-01-10T10:49:50	2025-01-10T10:56:50
git_branch	master	andrea.marziali/javacompenv
git_commit_date	1736503404	1736505492
git_commit_sha	`22458b3`	`8760af6`
release_version	1.46.0-SNAPSHOT~22458b3367	1.46.0-SNAPSHOT~8760af662e
start_time	2025-01-10T10:49:36	2025-01-10T10:56:36

See matching parameters

	Baseline	Candidate
application	insecure-bank	insecure-bank
ci_job_date	1736506964	1736506964
ci_job_id	761595378	761595378
ci_pipeline_id	52647645	52647645
cpu_model	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
variant	iast	iast

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics.

Request duration reports for petclinic

gantt title petclinic - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~8760af662e, baseline=1.46.0-SNAPSHOT~22458b3367 dateFormat X axisFormat %s section baseline no_agent (1.352 ms) : 1333, 1371 . : milestone, 1352, appsec (1.75 ms) : 1726, 1773 . : milestone, 1750, appsec_no_iast (1.756 ms) : 1732, 1781 . : milestone, 1756, iast (1.493 ms) : 1471, 1516 . : milestone, 1493, profiling (1.55 ms) : 1524, 1576 . : milestone, 1550, tracing (1.495 ms) : 1471, 1519 . : milestone, 1495, section candidate no_agent (1.353 ms) : 1332, 1373 . : milestone, 1353, appsec (1.753 ms) : 1729, 1776 . : milestone, 1753, appsec_no_iast (1.762 ms) : 1738, 1786 . : milestone, 1762, iast (1.513 ms) : 1491, 1535 . : milestone, 1513, profiling (1.517 ms) : 1491, 1542 . : milestone, 1517, tracing (1.483 ms) : 1458, 1508 . : milestone, 1483,

baseline results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	1.352 ms [1.333 ms, 1.371 ms]	-
appsec	1.75 ms [1.726 ms, 1.773 ms]	397.813 µs (29.4%)
appsec_no_iast	1.756 ms [1.732 ms, 1.781 ms]	404.212 µs (29.9%)
iast	1.493 ms [1.471 ms, 1.516 ms]	141.143 µs (10.4%)
profiling	1.55 ms [1.524 ms, 1.576 ms]	198.27 µs (14.7%)
tracing	1.495 ms [1.471 ms, 1.519 ms]	143.295 µs (10.6%)

candidate results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	1.353 ms [1.332 ms, 1.373 ms]	-
appsec	1.753 ms [1.729 ms, 1.776 ms]	400.273 µs (29.6%)
appsec_no_iast	1.762 ms [1.738 ms, 1.786 ms]	409.52 µs (30.3%)
iast	1.513 ms [1.491 ms, 1.535 ms]	160.319 µs (11.9%)
profiling	1.517 ms [1.491 ms, 1.542 ms]	163.944 µs (12.1%)
tracing	1.483 ms [1.458 ms, 1.508 ms]	130.594 µs (9.7%)

Request duration reports for insecure-bank

gantt title insecure-bank - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~8760af662e, baseline=1.46.0-SNAPSHOT~22458b3367 dateFormat X axisFormat %s section baseline no_agent (375.877 µs) : 356, 396 . : milestone, 376, iast (497.975 µs) : 476, 520 . : milestone, 498, iast_FULL (654.463 µs) : 633, 676 . : milestone, 654, iast_GLOBAL (523.144 µs) : 501, 545 . : milestone, 523, iast_HARDCODED_SECRET_DISABLED (509.807 µs) : 488, 531 . : milestone, 510, iast_INACTIVE (451.95 µs) : 431, 473 . : milestone, 452, iast_TELEMETRY_OFF (485.844 µs) : 464, 508 . : milestone, 486, tracing (452.364 µs) : 432, 473 . : milestone, 452, section candidate no_agent (387.029 µs) : 367, 407 . : milestone, 387, iast (495.705 µs) : 474, 517 . : milestone, 496, iast_FULL (656.797 µs) : 635, 678 . : milestone, 657, iast_GLOBAL (528.483 µs) : 506, 551 . : milestone, 528, iast_HARDCODED_SECRET_DISABLED (496.953 µs) : 475, 519 . : milestone, 497, iast_INACTIVE (452.06 µs) : 431, 473 . : milestone, 452, iast_TELEMETRY_OFF (486.401 µs) : 465, 508 . : milestone, 486, tracing (450.263 µs) : 429, 471 . : milestone, 450,

baseline results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	375.877 µs [355.83 µs, 395.924 µs]	-
iast	497.975 µs [475.682 µs, 520.268 µs]	122.098 µs (32.5%)
iast_FULL	654.463 µs [632.676 µs, 676.25 µs]	278.587 µs (74.1%)
iast_GLOBAL	523.144 µs [501.322 µs, 544.965 µs]	147.267 µs (39.2%)
iast_HARDCODED_SECRET_DISABLED	509.807 µs [488.314 µs, 531.3 µs]	133.93 µs (35.6%)
iast_INACTIVE	451.95 µs [431.197 µs, 472.702 µs]	76.073 µs (20.2%)
iast_TELEMETRY_OFF	485.844 µs [464.049 µs, 507.638 µs]	109.967 µs (29.3%)
tracing	452.364 µs [431.762 µs, 472.966 µs]	76.488 µs (20.3%)

candidate results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	387.029 µs [366.691 µs, 407.367 µs]	-
iast	495.705 µs [474.028 µs, 517.382 µs]	108.676 µs (28.1%)
iast_FULL	656.797 µs [635.213 µs, 678.382 µs]	269.768 µs (69.7%)
iast_GLOBAL	528.483 µs [506.037 µs, 550.93 µs]	141.454 µs (36.5%)
iast_HARDCODED_SECRET_DISABLED	496.953 µs [475.368 µs, 518.537 µs]	109.924 µs (28.4%)
iast_INACTIVE	452.06 µs [431.081 µs, 473.039 µs]	65.03 µs (16.8%)
iast_TELEMETRY_OFF	486.401 µs [464.929 µs, 507.872 µs]	99.371 µs (25.7%)
tracing	450.263 µs [429.363 µs, 471.163 µs]	63.234 µs (16.3%)

Dacapo

Parameters

	Baseline	Candidate
baseline_or_candidate	baseline	candidate
git_branch	master	andrea.marziali/javacompenv
git_commit_date	1736503404	1736505492
git_commit_sha	`22458b3`	`8760af6`
release_version	1.46.0-SNAPSHOT~22458b3367	1.46.0-SNAPSHOT~8760af662e

See matching parameters

	Baseline	Candidate
application	biojava	biojava
ci_job_date	1736507520	1736507520
ci_job_id	761595381	761595381
ci_pipeline_id	52647645	52647645
cpu_model	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
variant	appsec	appsec

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics.

Execution time for tomcat

gantt title tomcat - execution time [CI 0.99] : candidate=1.46.0-SNAPSHOT~8760af662e, baseline=1.46.0-SNAPSHOT~22458b3367 dateFormat X axisFormat %s section baseline no_agent (1.467 ms) : 1456, 1479 . : milestone, 1467, appsec (2.35 ms) : 2307, 2392 . : milestone, 2350, iast (2.097 ms) : 2043, 2151 . : milestone, 2097, iast_GLOBAL (2.141 ms) : 2087, 2195 . : milestone, 2141, profiling (1.948 ms) : 1906, 1991 . : milestone, 1948, tracing (1.936 ms) : 1894, 1977 . : milestone, 1936, section candidate no_agent (1.472 ms) : 1460, 1483 . : milestone, 1472, appsec (2.348 ms) : 2306, 2391 . : milestone, 2348, iast (2.11 ms) : 2056, 2164 . : milestone, 2110, iast_GLOBAL (2.139 ms) : 2085, 2194 . : milestone, 2139, profiling (1.973 ms) : 1930, 2017 . : milestone, 1973, tracing (1.944 ms) : 1902, 1986 . : milestone, 1944,

baseline results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	1.467 ms [1.456 ms, 1.479 ms]	-
appsec	2.35 ms [2.307 ms, 2.392 ms]	882.352 µs (60.1%)
iast	2.097 ms [2.043 ms, 2.151 ms]	629.677 µs (42.9%)
iast_GLOBAL	2.141 ms [2.087 ms, 2.195 ms]	673.595 µs (45.9%)
profiling	1.948 ms [1.906 ms, 1.991 ms]	480.921 µs (32.8%)
tracing	1.936 ms [1.894 ms, 1.977 ms]	468.237 µs (31.9%)

candidate results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	1.472 ms [1.46 ms, 1.483 ms]	-
appsec	2.348 ms [2.306 ms, 2.391 ms]	876.811 µs (59.6%)
iast	2.11 ms [2.056 ms, 2.164 ms]	638.222 µs (43.4%)
iast_GLOBAL	2.139 ms [2.085 ms, 2.194 ms]	667.455 µs (45.4%)
profiling	1.973 ms [1.93 ms, 2.017 ms]	501.824 µs (34.1%)
tracing	1.944 ms [1.902 ms, 1.986 ms]	472.508 µs (32.1%)

Execution time for biojava

gantt title biojava - execution time [CI 0.99] : candidate=1.46.0-SNAPSHOT~8760af662e, baseline=1.46.0-SNAPSHOT~22458b3367 dateFormat X axisFormat %s section baseline no_agent (15.366 s) : 15366000, 15366000 . : milestone, 15366000, appsec (15.161 s) : 15161000, 15161000 . : milestone, 15161000, iast (18.675 s) : 18675000, 18675000 . : milestone, 18675000, iast_GLOBAL (18.201 s) : 18201000, 18201000 . : milestone, 18201000, profiling (14.906 s) : 14906000, 14906000 . : milestone, 14906000, tracing (14.977 s) : 14977000, 14977000 . : milestone, 14977000, section candidate no_agent (15.457 s) : 15457000, 15457000 . : milestone, 15457000, appsec (14.877 s) : 14877000, 14877000 . : milestone, 14877000, iast (19.106 s) : 19106000, 19106000 . : milestone, 19106000, iast_GLOBAL (18.009 s) : 18009000, 18009000 . : milestone, 18009000, profiling (15.064 s) : 15064000, 15064000 . : milestone, 15064000, tracing (15.002 s) : 15002000, 15002000 . : milestone, 15002000,

baseline results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	15.366 s [15.366 s, 15.366 s]	-
appsec	15.161 s [15.161 s, 15.161 s]	-205.0 ms (-1.3%)
iast	18.675 s [18.675 s, 18.675 s]	3.309 s (21.5%)
iast_GLOBAL	18.201 s [18.201 s, 18.201 s]	2.835 s (18.4%)
profiling	14.906 s [14.906 s, 14.906 s]	-460.0 ms (-3.0%)
tracing	14.977 s [14.977 s, 14.977 s]	-389.0 ms (-2.5%)

candidate results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	15.457 s [15.457 s, 15.457 s]	-
appsec	14.877 s [14.877 s, 14.877 s]	-580.0 ms (-3.8%)
iast	19.106 s [19.106 s, 19.106 s]	3.649 s (23.6%)
iast_GLOBAL	18.009 s [18.009 s, 18.009 s]	2.552 s (16.5%)
profiling	15.064 s [15.064 s, 15.064 s]	-393.0 ms (-2.5%)
tracing	15.002 s [15.002 s, 15.002 s]	-455.0 ms (-2.9%)

PerfectSlayer

Few minor comments

internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java

internal-api/build.gradle

dd-trace-core/src/main/java/datadog/trace/core/DDSpanContext.java

dd-java-agent/instrumentation/wildfly-9/src/test/java/test/ModulePatchInstrumentation.java

PerfectSlayer

Looks good from platform 👍 You might want to have some eyes from IDM too 😉

dd-java-agent/instrumentation/wildfly-9/src/test/java/test/ModulePatchInstrumentation.java

mcculls · 2025-01-08T10:56:39Z

internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java

+
+ public static class ContextualInfo {
+ private final String serviceName;
+ private final Map<String, Object> tags = new HashMap<>();


Can tags be added concurrently by different threads, or added by one thread while queried by another?

If so then we'll either need a concurrent collection here, or add synchronization to methods accessing tags.

This is not happening since there is 1 deployment per 1 classloader so it cannot happen. For this reason I preferred to use a regular map

internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java

mcculls · 2025-01-08T11:13:09Z

internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java

+ return;
+ }
+ if (CAN_SPLIT_SERVICE_NAME_BY_DEPLOYMENT
+ && contextualInfo.serviceName != null


small inconsistency: here we're accessing the field, while the line below we're using the getter

also we access it several times in the same method, so consider pulling it into a local variable

I extracted a local var. It should be better now

...in/java/datadog/trace/instrumentation/wildfly/ResourceReferenceProcessorInstrumentation.java

mcculls · 2025-01-08T11:33:20Z

dd-trace-core/src/main/java/datadog/trace/core/CoreTracer.java

 serviceName = traceConfig.getPreferredServiceName();
 }
+ ClassloaderConfigurationOverrides.ContextualInfo contextualInfo =
+ ClassloaderConfigurationOverrides.maybeGetContextualInfo();


Previously we only called ClassloaderServiceNames when the span or its parent didn't have a service name.

We're now calling maybeGetContextualInfo unconditionally for every span which involves:

a call to Thread.currentThread().getContextClassLoader()

a call to WeakHashMap.get

Do we know how much overhead this could add to span creation? (a microbenchmark would be very helpful)

I added a benchmark. Even if it can be quite little overhead, as discussed, I put a boolean flag to act as a no-op when the map is empty saving the extra computation in the general non jee case

mcculls · 2025-01-08T11:46:53Z

internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java

+ private static final Function<ClassLoader, ContextualInfo> EMPTY_CONTEXTUAL_INFO_ADDER =
+ ignored -> new ContextualInfo(null);
+
+ private final WeakHashMap<ClassLoader, ContextualInfo> weakCache = new WeakHashMap<>();


WeakHashMap is not a concurrent collection - to be safe we should really wrap this in synchronizedMap, but that would have performance implications given we're now accessing the map for every span.

There is WeakConcurrentMap but we cannot use that directly in internal-api because it would leak that implementation onto the application class-path. You might therefore need to move this cache to inside dd-trace-core or somewhere similarly isolated.

@mcculls thanks. I protected the write path with a lock. The read path is not needing to be thread safe. There could be parallel deployment needing to write the map. However the read are usually happening afterwards.
I also added a quick optimization using a volatile flag in order to skip any costly operation when the map is just empty

mcculls

Just one concern - we're now accessing the weak cache for every span regardless whether this feature is enabled or not. The current implementation used for the weak cache is not thread-safe and should really be wrapped in synchronizedMap.

Given this I'd like to see some microbenchmarking done to check how much overhead this will add. (We also need to resolve whether synchronizedMap is OK or should we consider using WeakConcurrentMap indirectly - benchmarking will also help with that)

ygree

LGTM

…urationOverrides.java Co-authored-by: Bruce Bujon <PerfectSlayer@users.noreply.github.com>

…urationOverrides.java Co-authored-by: Stuart McCulloch <stuart.mcculloch@datadoghq.com>

…enabled

* Use env-entry to add tags per webapp deployment * fix gradle file * Migrate to hasmethodadvice * exclude classes from coverage * codenarc * add more repos * jacoco * Update internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java Co-authored-by: Bruce Bujon <PerfectSlayer@users.noreply.github.com> * review * use our named * more coverage * Update internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java Co-authored-by: Stuart McCulloch <stuart.mcculloch@datadoghq.com> * Update internal-api/src/main/java/datadog/trace/api/ClassloaderConfigurationOverrides.java Co-authored-by: Stuart McCulloch <stuart.mcculloch@datadoghq.com> * review * add jmh * optimize * widen muzzle excludes * exclude lazy from branch coverage * clean * Do not set contextual service name if jee-split-by-deployment is not enabled --------- Co-authored-by: Bruce Bujon <PerfectSlayer@users.noreply.github.com> Co-authored-by: Stuart McCulloch <stuart.mcculloch@datadoghq.com>

| Package | Type | Package file | Manager | Update | Change | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-datastore](https://github.com/googleapis/java-datastore) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `2.25.4` -> `2.26.0` | | [com.datadoghq:dd-trace-api](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.45.2` -> `1.46.0` | | [com.datadoghq:dd-trace-ot](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.45.2` -> `1.46.0` | | [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | --- ### Release Notes <details> <summary>googleapis/java-datastore (com.google.cloud:google-cloud-datastore)</summary> ### [`v2.26.0`](https://github.com/googleapis/java-datastore/blob/HEAD/CHANGELOG.md#2260-2025-01-29) ##### Features - Add firestoreInDatastoreMode for datastore emulator ([#1698](googleapis/java-datastore#1698)) ([50f106d](googleapis/java-datastore@50f106d)) ##### Dependencies - Update dependency com.google.cloud:sdk-platform-java-config to v3.42.0 ([#1725](googleapis/java-datastore#1725)) ([1cbaf22](googleapis/java-datastore@1cbaf22)) </details> <details> <summary>datadog/dd-trace-java (com.datadoghq:dd-trace-api)</summary> ### [`v1.46.0`](https://github.com/DataDog/dd-trace-java/releases/tag/v1.46.0): 1.46.0 ##### Breaking Changes > \[!WARNING] > jnr-unixsocket is now an external dependency of dd-trace-ot and must be included when deploying dd-trace-ot. > \[!NOTE] > The API `TracerScope.setAsync(boolean)`, used to manually control asynchronous span propagation, does no more apply to the scope instance but to the active span scope. ##### Components ##### Application Security Management (IAST) - 🐛 Fix String.replace instrumentation for IAST ([#8281](DataDog/dd-trace-java#8281) - [@Mariovido](https://github.com/Mariovido)) - ✨ Apply the standard nomenclature to the stacktrace configs ([#8244](DataDog/dd-trace-java#8244) - [@jandro996](https://github.com/jandro996)) - 🐛 Exclude false positive weak randomness ([#8232](DataDog/dd-trace-java#8232) - [@jandro996](https://github.com/jandro996)) - ✨ Propagation of translateEscapes of String class ([#8186](DataDog/dd-trace-java#8186) - [@sezen-datadog](https://github.com/sezen-datadog)) - ✨ Add security control metrics ([#8175](DataDog/dd-trace-java#8175) - [@jandro996](https://github.com/jandro996)) - ✨ Increase IAST propagation to StringBuffer setLength ([#8128](DataDog/dd-trace-java#8128) - [@Mariovido](https://github.com/Mariovido)) - ✨ Add IAST taint tracking for DB values ([#8072](DataDog/dd-trace-java#8072) - [@Mariovido](https://github.com/Mariovido)) ##### Application Security Management (WAF) - 🐛 Prevents a NPE when there is no subscriber for user events ([#8258](DataDog/dd-trace-java#8258) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - ✨ Apply the standard nomenclature to the stacktrace configs ([#8244](DataDog/dd-trace-java#8244) - [@jandro996](https://github.com/jandro996)) - 🐛 Ensure cached subscriptions are cleared on reconfiguration via RC ([#8229](DataDog/dd-trace-java#8229) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - ✨ Add support for session tracking in Vertx ([#8167](DataDog/dd-trace-java#8167) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - ✨ Create span tag: \_dd.appsec.rasp.timeout ([#8269](DataDog/dd-trace-java#8269) - [@Mariovido](https://github.com/Mariovido)) ##### Build & Tooling - 🐛 Ensure shaded helpers have unique names when injected into class-loaders ([#8192](DataDog/dd-trace-java#8192) - [@mcculls](https://github.com/mcculls)) ##### Configuration at Runtime - 🐛 Remove filtering of `DD_SERVICE` and `DD_ENV` from the tracer ([#8176](DataDog/dd-trace-java#8176) - [@mhlidd](https://github.com/mhlidd)) ##### Continuous Integration Visibility - 🧹 Generalize TestRetryPolicy to TestExecutionPolicy ([#8302](DataDog/dd-trace-java#8302) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Parallelize CI Visibility settings requests ([#8299](DataDog/dd-trace-java#8299) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Generalize test retry logic ([#8289](DataDog/dd-trace-java#8289) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Generalize tests skipping logic ([#8288](DataDog/dd-trace-java#8288) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Remove skip and shouldBeSkipped methods from TestEventsHandler in favor of isSkippable ([#8286](DataDog/dd-trace-java#8286) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨⚡ Optimize Git repository information computation ([#8270](DataDog/dd-trace-java#8270) - [@dougqh](https://github.com/dougqh)) - ✨ Always request known tests from the backend ([#8268](DataDog/dd-trace-java#8268) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨ Fix NPE when trying to get retry analyzer in Test NG ([#8253](DataDog/dd-trace-java#8253) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🐛 Set test framework and test framework version tags atomically ([#8252](DataDog/dd-trace-java#8252) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨ Add debug logging to Android Gradle module layout logic ([#8251](DataDog/dd-trace-java#8251) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🐛 Fix source and destination folders computation for Android Gradle projects ([#8190](DataDog/dd-trace-java#8190) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨ Add basic Scala Weaver sbt support ([#8189](DataDog/dd-trace-java#8189) - [@daniel-mohedano](https://github.com/daniel-mohedano)) - ✨ Implement impacted tests detection ([#8188](DataDog/dd-trace-java#8188) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) ##### Data Streams Monitoring - ✨ Change hash computation for protobuf to better represent impacting changes + save proto number in schema ([#8201](DataDog/dd-trace-java#8201) - [@vandonr](https://github.com/vandonr)) ##### Database Monitoring - Add peer service tag in dbm sql commenter ([#7913](DataDog/dd-trace-java#7913) - [@jordan-wong](https://github.com/jordan-wong)) ##### Dynamic Instrumentation - ✨ Add support for SymDB to scan directories ([#8306](DataDog/dd-trace-java#8306) - [@jpbempel](https://github.com/jpbempel)) - ✨ Add SymDB report for any jar scanning failures ([#8300](DataDog/dd-trace-java#8300) - [@jpbempel](https://github.com/jpbempel)) - ✨ Use two budgets depending on type ([#8283](DataDog/dd-trace-java#8283) - [@evanchooly](https://github.com/evanchooly)) - ✨ Institute a 10 snapshot per probe per trace budget ([#8277](DataDog/dd-trace-java#8277) - [@evanchooly](https://github.com/evanchooly)) - 🐛 Avoid double snapshots for Exception Replay ([#8273](DataDog/dd-trace-java#8273) - [@jpbempel](https://github.com/jpbempel)) - ✨ Simplify code origins. Separate out snapshot generation. ([#8263](DataDog/dd-trace-java#8263) - [@evanchooly](https://github.com/evanchooly)) - ✨ Add Exception probe custom instrumentation ([#8230](DataDog/dd-trace-java#8230) - [@jpbempel](https://github.com/jpbempel)) - ✨ Enhance log probes to honor debug session tags ([#8215](DataDog/dd-trace-java#8215) - [@evanchooly](https://github.com/evanchooly)) - 🐛 Don't redact env tokens from debugger probe snapshots ([#8211](DataDog/dd-trace-java#8211) - [@watson](https://github.com/watson)) - ✨⚡ Move Trace/SpanId capture at commit time ([#8184](DataDog/dd-trace-java#8184) - [@jpbempel](https://github.com/jpbempel)) - 🐛 Capture values at entry for method probe ([#8169](DataDog/dd-trace-java#8169) - [@jpbempel](https://github.com/jpbempel)) ##### JMX fetch - 🐛 Mute JMXFetch Shutdown in progress error ([#8068](DataDog/dd-trace-java#8068) - [@ygree](https://github.com/ygree)) ##### OpenTracing - ⚠️🧹 Make jnr-unixsocket an explicit dependency of dd-trace-ot ([#8307](DataDog/dd-trace-java#8307) - [@mcculls](https://github.com/mcculls)) ##### Profiling - 🐛 Avoid unsupported API call for creating folders on windows ([#8304](DataDog/dd-trace-java#8304) - [@jbachorik](https://github.com/jbachorik)) - ✨ Tag profiles for serverless ([#8279](DataDog/dd-trace-java#8279) - [@jbachorik](https://github.com/jbachorik)) - ✨ add queue type and length to queue events ([#8242](DataDog/dd-trace-java#8242) - [@richardstartin](https://github.com/richardstartin)) - 🐛 TempLocationManager Fixes and Improvements ([#8191](DataDog/dd-trace-java#8191) - [@jbachorik](https://github.com/jbachorik)) - ✨ Bump ddprof to 1.18.0 ([#8173](DataDog/dd-trace-java#8173) - [@jbachorik](https://github.com/jbachorik)) - ✨ Report profiler initialization and configuration errors to telemetry ([#8171](DataDog/dd-trace-java#8171) - [@jbachorik](https://github.com/jbachorik)) ##### Telemetry - ✨ Add pending traces report in tracer flares ([#8053](DataDog/dd-trace-java#8053) - [@mhlidd](https://github.com/mhlidd)) ##### Testing - ✨ Test http server requests in parallel ([#8222](DataDog/dd-trace-java#8222) - [@amarziali](https://github.com/amarziali)) ##### Trace context propagation - ✨ Add non default propagator registration ([#8310](DataDog/dd-trace-java#8310) - [@PerfectSlayer](https://github.com/PerfectSlayer)) ##### Tracer core - ✨ Probe for existence of IBMSASL or ACCP security providers ([#8276](DataDog/dd-trace-java#8276) - [@mcculls](https://github.com/mcculls)) - ✨⚡ Overhead improvement to agent feedback based sampling ([#8265](DataDog/dd-trace-java#8265) - [@dougqh](https://github.com/dougqh)) - 🧹 Move async propagation API from scope to tracer ([#8231](DataDog/dd-trace-java#8231) - [@PerfectSlayer](https://github.com/PerfectSlayer)) - ✨ Introduce context propagation API ([#8161](DataDog/dd-trace-java#8161) - [@PerfectSlayer](https://github.com/PerfectSlayer)) - ✨🧪 Use env-entry to add tags per webapp deployment ([#8138](DataDog/dd-trace-java#8138) - [@amarziali](https://github.com/amarziali)) - ✨ Introduce context helpers API ([#8134](DataDog/dd-trace-java#8134) - [@PerfectSlayer](https://github.com/PerfectSlayer)) - ✨ Support IPv6 values for `DD_AGENT_HOST` and `DD_TRACE_AGENT_URL` ([#7984](DataDog/dd-trace-java#7984) - [@mhlidd](https://github.com/mhlidd)) ##### Instrumentations ##### Apache HttpComponents - 🐛 Properly finish spans and support latest apache httpclient5 ([#8272](DataDog/dd-trace-java#8272) - [@amarziali](https://github.com/amarziali)) ##### AWS Lambda instrumentation - 🐛 Properly capture lambda payloads for all handler types. ([#8264](DataDog/dd-trace-java#8264) - [@purple4reina](https://github.com/purple4reina)) ##### AWS S3 instrumentation - 💡 Create S3 instrumentation + add span pointers ([#8075](DataDog/dd-trace-java#8075) - [@nhulston](https://github.com/nhulston)) ##### AWS SDK instrumentation - 🐛 Revert "Add avoid double instrumenting lambda non-streaming handlers." ([#8247](DataDog/dd-trace-java#8247) - [@nhulston](https://github.com/nhulston)) ##### Cassandra - ✨ Allow extracting keyspace from statement result ([#8239](DataDog/dd-trace-java#8239) - [@amarziali](https://github.com/amarziali)) ##### Core Java language instrumentation - ✨ Propagation of translateEscapes of String class ([#8186](DataDog/dd-trace-java#8186) - [@sezen-datadog](https://github.com/sezen-datadog)) ##### Eclipse Vert.x instrumentation - 🐛 Fix vertx worker propagation and error handling ([#8237](DataDog/dd-trace-java#8237) - [@amarziali](https://github.com/amarziali)) - ✨ Support vertx 5 ([#8220](DataDog/dd-trace-java#8220) - [@amarziali](https://github.com/amarziali)) - ✨ Add support for session tracking in Vertx ([#8167](DataDog/dd-trace-java#8167) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) ##### Kafka instrumentation - 🐛 Prevent possible NPE calculating Kafka record header size ([#8292](DataDog/dd-trace-java#8292) - [@ygree](https://github.com/ygree)) ##### Mule instrumentation - 🐛 Fix crash using Mule with JPMS ([#8187](DataDog/dd-trace-java#8187) - [@amarziali](https://github.com/amarziali)) ##### Protocol Buffer instrumentation - ✨ Change hash computation for protobuf to better represent impacting changes + save proto number in schema ([#8201](DataDog/dd-trace-java#8201) - [@vandonr](https://github.com/vandonr)) ##### Spring instrumentation - 🐛 Preserve getQualifier from spring scheduling runnables ([#8293](DataDog/dd-trace-java#8293) - [@amarziali](https://github.com/amarziali)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am every weekday" in timezone Australia/Melbourne, Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). GitOrigin-RevId: bb09d47e4eed77a003f630273b4d0a84003eb899

This reverts commit a3a8fe6.

amarziali added type: enhancement Enhancements and improvements comp: core Tracer core tag: experimental Experimental changes labels Dec 31, 2024

amarziali marked this pull request as ready for review December 31, 2024 14:30

amarziali requested review from a team as code owners December 31, 2024 14:30

amarziali requested a review from PerfectSlayer December 31, 2024 14:30

amarziali force-pushed the andrea.marziali/javacompenv branch from d3e1f2d to 62f2214 Compare January 2, 2025 08:50

PerfectSlayer reviewed Jan 2, 2025

View reviewed changes

amarziali force-pushed the andrea.marziali/javacompenv branch from 764661d to 8fce1cb Compare January 3, 2025 09:51

PerfectSlayer approved these changes Jan 6, 2025

View reviewed changes

dd-java-agent/instrumentation/wildfly-9/src/test/java/test/ModulePatchInstrumentation.java Show resolved Hide resolved

amarziali requested a review from mcculls January 7, 2025 10:53