Skip to content

ci(py3.15): add 3.15 to riot matrix with gated suites - #19253

Closed
vlad-scherbich wants to merge 7 commits into
gab/315-wrapping-contextfrom
vlad/315-ci-matrix
Closed

ci(py3.15): add 3.15 to riot matrix with gated suites#19253
vlad-scherbich wants to merge 7 commits into
gab/315-wrapping-contextfrom
vlad/315-ci-matrix

Conversation

@vlad-scherbich

@vlad-scherbich vlad-scherbich commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

prev: #17849 | next: #19252

Summary

Riotfile 3.15, docker testrunner, suite gating.

Test plan

  • CI green on this branch
  • Stack merges cleanly into the next PR's base branch

@datadog-datadog-prod-us1-2

datadog-datadog-prod-us1-2 Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Pipelines  Tests

Unblock PR with BitsAI

⚠️ Warnings

🚦 4 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-py | check_requirements_lockfiles   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-py | baseline:build   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-py | build_base_venvs: [3.15]   View in Datadog   GitLab

View all 4 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🔄 Datadog auto-retried 2 jobs - 1 passed on retry View in Datadog

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 3980414 | Docs | Datadog PR Page | Give us feedback!

@vlad-scherbich
vlad-scherbich changed the base branch from main to gab/315-wrapping-context July 23, 2026 14:18
@vlad-scherbich
vlad-scherbich force-pushed the gab/315-wrapping-context branch from fcb07a1 to 43c73c5 Compare July 23, 2026 14:31
@vlad-scherbich
vlad-scherbich force-pushed the vlad/315-ci-matrix branch 2 times, most recently from 3897d97 to f0ee2dd Compare July 23, 2026 14:31
@pr-commenter

pr-commenter Bot commented Jul 23, 2026

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2026-07-23 14:42:31

Comparing candidate commit 3897d97 in PR branch vlad/315-ci-matrix with baseline commit bd180ec in branch main.

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

scenario:iastaspects-lstrip_aspect

  • 🟥 execution_time [+59.028µs; +63.745µs] or [+18.571%; +20.054%]

scenario:iastaspects-translate_aspect

  • 🟥 execution_time [+41.572µs; +49.328µs] or [+8.349%; +9.907%]

scenario:iastaspectsospath-ospathbasename_aspect

  • 🟥 execution_time [+84.487µs; +92.627µs] or [+19.805%; +21.713%]

scenario:span-start

  • 🟥 execution_time [+1.130ms; +1.300ms] or [+7.133%; +8.205%]

scenario:telemetryaddmetric-1-count-metric-1-times

  • 🟥 execution_time [+186.191ns; +231.514ns] or [+8.672%; +10.782%]

faydef and others added 5 commits July 23, 2026 15:15
## Description

#19217 added a temporary `image: registry.ddbuild.io/images/benchmarking-platform-tools-ubuntu:6845f3c7` override to the `notify-slo-breaches` job to work around a `latest`-tag denial on `check-slo-breaches`. That override image doesn't carry the dependencies `notify-slo-breaches` needs (e.g. `postmessage`), so the job now fails:

```
postmessage: command not found
```
https://gitlab.ddbuild.io/DataDog/apm-reliability/dd-trace-py/-/jobs/1885560137

This reverts the override on `notify-slo-breaches` only. `check-slo-breaches` (both macro- and micro-benchmarks) keeps its pin, since that's the job the original fix targeted and isn't affected by this failure.

## Testing

CI: the next pipeline's `notify-slo-breaches` job should run with the default (template-provided) image and no longer fail on `postmessage`.

## Risks

None — reverts to the job's previous, working image resolution.

## Additional Notes

None

Co-authored-by: fayssal.defaa <fayssal.defaa@datadoghq.com>
@vlad-scherbich vlad-scherbich added the changelog/no-changelog A changelog entry is not required for this PR. label Jul 23, 2026
@vlad-scherbich
vlad-scherbich requested review from a team and kathiehuang and removed request for a team July 23, 2026 16:14
@cit-pr-commenter-54b7da

Copy link
Copy Markdown

Codeowners resolved as

tests/internal/test_auto.py                                             @DataDog/apm-core-python
tests/internal/test_forksafe.py                                         @DataDog/apm-core-python
tests/internal/test_safety.py                                           @DataDog/apm-core-python

@cit-pr-commenter-54b7da

Copy link
Copy Markdown

Circular import analysis

⚠️ Existing circular imports

There are 33 circular imports that already exist on the base branch and have not been changed by this PR.

Show existing cycles (showing 5 of 33 shortest)
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.kafka -> ddtrace.internal.datastreams
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.botocore -> ddtrace.internal.datastreams
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.kombu -> ddtrace.internal.datastreams
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.google_cloud_pubsub -> ddtrace.internal.datastreams
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.aiokafka -> ddtrace.internal.datastreams

To see all cycles, download the cycles-base.json and cycles-pr.json artifacts from this CI job and run:

uv run --script scripts/import-analysis/cycles.py compare cycles-base.json cycles-pr.json

@vlad-scherbich vlad-scherbich changed the title Vlad/315 ci matrix ci(py3.15): add 3.15 to riot matrix with gated suites Jul 23, 2026
@vlad-scherbich
vlad-scherbich deleted the branch gab/315-wrapping-context July 24, 2026 02:07
@vlad-scherbich

Copy link
Copy Markdown
Contributor Author

Superseded: the profiling stack (#19267#19275, tip #19254) is rebased onto #19724 (vlad/ci-fix-smoke-tests-py315). This PR is no longer in the stack chain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog A changelog entry is not required for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants