Skip to content

Auto-activate java easy wins for apm-lang-platform #7515

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
dd-octo-sts wants to merge 1 commit into
base: main
Choose a base branch
from
+9 −5
Open

Auto-activate java easy wins for apm-lang-platform #7515

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 9 additions & 5 deletions manifests/java.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4065,7 +4065,7 @@ manifest:
- declaration: missing_feature (OTel resource naming implemented in 1.24.0)
component_version: <=1.23.0
tests/parametric/test_otel_tracestate_sampling.py::Test_OtelTracestateSampling: missing_feature (APMAPI-2171)
tests/parametric/test_otlp_trace_metrics.py: v1.66.0-SNAPSHOT
tests/parametric/test_otlp_trace_metrics.py: v1.66.0-SNAPSHOT # Easy win for all weblogs and version 1.65.1
tests/parametric/test_otlp_trace_metrics.py::Test_FR01_Enablement_Configuration::test_fr01_6_disabled_when_apm_tracing_disabled: missing_feature (OTLP trace metrics are still emitted when DD_APM_TRACING_ENABLED=false)
tests/parametric/test_otlp_trace_metrics.py::Test_FR08_AdditionalTags::test_fr08_10_dd_tags_tracer_tags_resource_attributes: missing_feature (DD_TAGS is not yet reported as datadog.tracer_tags on the OTLP trace metrics resource)
tests/parametric/test_otlp_trace_metrics.py::Test_FR08_AdditionalTags::test_fr08_11_otel_resource_attributes_env_tracer_tags: missing_feature (OTEL_RESOURCE_ATTRIBUTES is not yet reported as datadog.tracer_tags on the OTLP trace metrics resource)
Expand All @@ -4086,8 +4086,12 @@ manifest:
tests/parametric/test_parametric_endpoints.py::Test_Parametric_Write_Log: missing_feature
tests/parametric/test_parametric_endpoints.py::Test_Parametric_Write_Log::test_create_logger: incomplete_test_app (Logs endpoint is only implemented in python and node.js app)
tests/parametric/test_parametric_endpoints.py::Test_Parametric_Write_Log::test_write_log: incomplete_test_app (Logs endpoint is only implemented in python and node.js app)
tests/parametric/test_partial_flushing.py::Test_Partial_Flushing::test_partial_flushing_one_span: 'missing_feature (java uses ''>'' so it needs one more span to force a partial flush)'
tests/parametric/test_partial_flushing.py::Test_Partial_Flushing::test_partial_flushing_one_span_default: 'missing_feature (java uses ''>'' so it needs one more span to force a partial flush)'
tests/parametric/test_partial_flushing.py::Test_Partial_Flushing::test_partial_flushing_one_span: # TODO: a lower version might be supported
- declaration: missing_feature (java uses '>' so it needs one more span to force a partial flush)
component_version: <1.65.1
tests/parametric/test_partial_flushing.py::Test_Partial_Flushing::test_partial_flushing_one_span_default: # TODO: a lower version might be supported
- declaration: missing_feature (java uses '>' so it needs one more span to force a partial flush)
component_version: <1.65.1
Comment on lines +4091 to +4094

@chatgpt-codex-connector chatgpt-codex-connector Bot Aug 14, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Quote the new version ranges

For these two new manifest clauses, component_version starts with < but is left as an unquoted plain scalar. The PR review rules require YAML values containing < to be quoted, so these should be '<1.65.1'; otherwise future formatting/validation can disagree with the documented manifest syntax. .cursor/rules/pr-review.mdcL61-L65

Useful? React with 👍 / 👎.

tests/parametric/test_partial_flushing.py::Test_Partial_Flushing::test_partial_flushing_propagation_tags: v1.65.0-SNAPSHOT+7071ae2ed6
tests/parametric/test_process_discovery.py::Test_ProcessDiscovery: v1.55.0-SNAPSHOT
tests/parametric/test_sampling_manual.py::Test_Manual_Sampling: v1.59.0-SNAPSHOT+419da213f7
Expand Down Expand Up @@ -4264,8 +4268,8 @@ manifest:
tests/stats/test_stats.py::Test_Error_Sampler:
- weblog_declaration:
"*": v1.54.0
vertx3: missing_feature (500 responses are not flagged as errors, so no error trace is produced)
vertx4: missing_feature (500 responses are not flagged as errors, so no error trace is produced)
vertx3: v1.65.1 # TODO: a lower version might be supported
vertx4: v1.65.1 # TODO: a lower version might be supported
tests/stats/test_stats.py::Test_Stats_Service_Source:
- weblog_declaration:
"*": v0.0.0
Expand Down
Loading