You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
otelbot/draft-release-notes-36949722178open-telemetry/opentelemetry-java-instrumentation:otelbot/draft-release-notes-36949722178Copy head branch name to clipboard
otelbot/draft-release-notes-36949722178open-telemetry/opentelemetry-java-instrumentation:otelbot/draft-release-notes-36949722178Copy head branch name to clipboard
Automated draft of the ## Unreleased section of CHANGELOG.md.
The entire ## Unreleased block was regenerated from per-PR classifications. Existing entries are intentionally replaced so the generated draft can be reviewed against them.
Document deprecation of the entire Elasticsearch artifact
CHANGELOG.md:54
[General] The published opentelemetry-elasticsearch-rest-7.0 artifact is also deprecated, not just these two entrypoint classes (instrumentation/elasticsearch/README.md:3-8). Omitting the artifact-level deprecation leaves dependency consumers unaware that the entire library artifact is scheduled for removal.
Clarify sampler deprecation scope and SDK initialization failure
CHANGELOG.md:101
[Config] This deprecation is limited to the samplers bundled with the Java agent and Spring Boot starter, and #20249 also makes selecting either one fail SDK initialization under v3 preview. The unqualified wording overstates the affected surface while omitting the immediate failure mode users must address before enabling preview.
Document deprecation and behavior change for search query capture
CHANGELOG.md:170
[Config] #19675 also deprecates the stable otel.instrumentation.elasticsearch.capture-search-query setting (and YAML java.elasticsearch.capture_search_query) with no replacement; the implementation now warns that it will be removed. This enhancement entry does not communicate that migration impact, so add a separate bullet under Deprecations explaining that query capture becomes unconditional while sanitization remains configurable.
Document required isRequestStreaming API change
CHANGELOG.md:274
[General] This records the new attribute but omits #19879's binary-incompatible API change: GenAiAttributesGetter gained the required isRequestStreaming(REQUEST) method. Existing implementations can fail with AbstractMethodError when the extractor invokes it, so add a separate bullet under Breaking changes telling implementers to add the method.
Classify host.id default removal as a breaking change
CHANGELOG.md:366
[General] This changes the Java agent default so existing users stop emitting host.id unless they opt in. The release-note rules classify unconditional emitted-attribute changes as Breaking (.github/scripts/draft-release-notes/rules.md:57-63), so placing this under Enhancements hides an upgrade-impacting telemetry removal; move it to the appropriate Breaking section.
Specify the exact opt-in value for the preview flag
CHANGELOG.md:398
[Config] This omits the enabling value for the boolean preview flag, unlike the surrounding gated entries. As written it can be read as applying whenever the property is present, including when set to false; name the exact opt-in value.
Clarify Elasticsearch REST library artifacts are deprecated
CHANGELOG.md:54
[General] This narrows the deprecation to two entry-point classes, but the Elasticsearch REST library artifacts themselves are deprecated. Restore the artifact-level wording so consumers understand that the dependency, not merely these two types, is being retired.
Document deprecation of Elasticsearch REST library artifact
CHANGELOG.md:51
[General] The deprecation covers the published Elasticsearch REST library artifact as well as these two entry points. Omitting the artifact makes it appear that consumers can continue using another API from the library, even though the artifact itself is scheduled for removal.
The reason will be displayed to describe this comment to others. Learn more.
Copilot review overview
🔵 Needs a closer look
Several migration notes omit compatibility details needed to avoid incorrect configuration or telemetry expectations.
Review effort: Balanced Findings: None
Previously missed (3)
In code that hasn't changed since last review
Clarify metric pattern migration and automatic stable metrics
CHANGELOG.md:23
[Config] The old value is a target name such as tomcat, while the replacement accepts metric-name patterns such as tomcat.*; copying the value across would select no unstable metrics. Also clarify that stable metrics are collected automatically so the migration is actionable.
[General] The recommended native client support is only available in Elasticsearch Java API Client 7.17.20+ on the 7.x line and 8.10+. Without those bounds, users on earlier client versions are directed to functionality their dependency does not provide.
Clarify JVM metrics differ from deprecated OSHI process metrics
CHANGELOG.md:80
[General] These JVM metrics are not equivalent replacements for the deprecated OSHI process metrics: jvm.memory.used reports JVM memory pools rather than process RSS/VMS, and jvm.cpu.time does not preserve the user/system split. Restore that distinction so users do not expect equivalent telemetry after migration.
[Config] This migration is not effective in the release's default mode: JmxMetricInsightInstaller reads java.jmx.enabled (bridged to otel.jmx.enabled) outside v3-preview, while otel.instrumentation.jmx.enabled is consulted only in the v3-preview branch. A user following this note can set the replacement to false and still get JMX metrics. Clarify that the old key remains required in the current default mode and that the new key is the 3.0 replacement.
The reason will be displayed to describe this comment to others. Learn more.
Copilot review overview
🔵 Needs a closer look
Two deprecation entries need more accurate migration guidance.
Review effort: Balanced Findings: None
Previously missed (2)
In code that hasn't changed since last review
Clarify otel.jmx.target.system to metric-name glob migration
CHANGELOG.md:23
[General] otel.jmx.target.system accepts target names and enables both stable and unstable definitions, whereas the replacement accepts metric-name glob patterns and only opts in unstable bundled metrics; stable bundled metrics are automatic. Clarify that migration so users do not copy values such as tomcat into the new property (the working equivalent is tomcat.*).
[General] The two JVM metrics are not equivalent replacements for ProcessMetrics: they drop process RSS/virtual-memory measurements and the user/system CPU-time split. Preserve this migration caveat so users do not assume their existing process-metric dashboards can switch names without losing data.
The reason will be displayed to describe this comment to others. Learn more.
Copilot review overview
🔵 Needs a closer look
The OSHI migration note omits a consequential difference between the deprecated metrics and their suggested replacements.
Review effort: Balanced Findings: None
Previously missed (1)
In code that hasn't changed since last review
Preserve warning that JVM metrics do not replace ProcessMetrics
CHANGELOG.md:79
[General] This reads as though the JVM metrics are equivalent replacements, but ProcessMetrics records process RSS/virtual memory and separate user/system CPU times, while the suggested JVM metrics do not. Preserve the non-equivalence warning from the API Javadoc so users do not unknowingly lose those measurements during migration.
The reason will be displayed to describe this comment to others. Learn more.
Copilot review overview
🔵 Needs a closer look
Two deprecation entries need accurate migration details.
Review effort: Balanced Findings: None
Previously missed (2)
In code that hasn't changed since last review
Document migration from target names to metric-name glob patterns
CHANGELOG.md:22
[Config] The replacement property selects metric-name glob patterns, not target-system names, so this reads like a direct property rename even though reusing a value such as tomcat would not enable the target's metrics. Include the value migration (for example, tomcat → tomcat.*) so users can preserve their collection.
Document minimum client versions for native OpenTelemetry support
CHANGELOG.md:40
[General] Native OpenTelemetry support is only available in Elasticsearch Java API Client 7.17.20+ on the 7.x line and 8.10+. Omitting these minimum versions directs users of earlier clients to a feature they do not have; preserve the version-qualified migration guidance used by the deprecated API's Javadoc.
The reason will be displayed to describe this comment to others. Learn more.
Copilot review overview
🔵 Needs a closer look
The structured-logging entry omits four deprecated configuration properties.
Review effort: Balanced Findings: None
Previously missed (1)
In code that hasn't changed since last review
Document migration from deprecated capture-* boolean properties
CHANGELOG.md:54
[Config] This documents only the source-specific .included/.excluded selectors, but #20066 also deprecates the four capture-* boolean properties. Users relying on those toggles receive no migration guidance; include them here and direct them to the unified .included selector (* preserves the enabled behavior).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated draft of the
## Unreleasedsection ofCHANGELOG.md.The entire
## Unreleasedblock was regenerated from per-PR classifications. Existing entries are intentionally replaced so the generated draft can be reviewed against them.Download draft diagnostics