dd-trace-py icon indicating copy to clipboard operation
dd-trace-py copied to clipboard

chore(ci): riotize appsec_integrations_django

Open juanjux opened this issue 5 months ago • 3 comments

22 sub-steps x 5 minutes less = 110 less CPU minutes per run.

Signed-off-by: Juanjo Alvarez [email protected]## Checklist

  • [X] PR author has checked that all the criteria below are met
  • The PR description includes an overview of the change
  • The PR description articulates the motivation for the change
  • The change includes tests OR the PR description describes a testing strategy
  • The PR description notes risks associated with the change, if any
  • Newly-added code is easy to change
  • The change follows the library release note guidelines
  • The change includes or references documentation updates if necessary
  • Backport labels are set (if applicable)

Reviewer Checklist

  • [ ] Reviewer has checked that all the criteria below are met
  • Title is accurate
  • All changes are related to the pull request's stated goal
  • Avoids breaking API changes
  • Testing strategy adequately addresses listed risks
  • Newly-added code is easy to change
  • Release note makes sense to a user of the library
  • If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment
  • Backport labels are set in a manner that is consistent with the release branch maintenance policy

juanjux avatar Jun 26 '25 09:06 juanjux

CODEOWNERS have been resolved as:

.riot/requirements/18114ab.txt                                          @DataDog/apm-python
.riot/requirements/4c5069a.txt                                          @DataDog/apm-python
.riot/requirements/bc81785.txt                                          @DataDog/apm-python
.riot/requirements/c584bb3.txt                                          @DataDog/apm-python
ddtrace/appsec/_iast/_handlers.py                                       @DataDog/asm-python
hatch.toml                                                              @DataDog/python-guild
riotfile.py                                                             @DataDog/apm-python
tests/appsec/suitespec.yml                                              @DataDog/asm-python
.riot/requirements/100a87e.txt                                          @DataDog/apm-python
.riot/requirements/10a2bf0.txt                                          @DataDog/apm-python
.riot/requirements/1337ee3.txt                                          @DataDog/apm-python
.riot/requirements/1407803.txt                                          @DataDog/apm-python
.riot/requirements/14b54db.txt                                          @DataDog/apm-python
.riot/requirements/15942cc.txt                                          @DataDog/apm-python
.riot/requirements/15c4ab2.txt                                          @DataDog/apm-python
.riot/requirements/18f48b8.txt                                          @DataDog/apm-python
.riot/requirements/1b45436.txt                                          @DataDog/apm-python
.riot/requirements/1ba7be6.txt                                          @DataDog/apm-python
.riot/requirements/1e62aea.txt                                          @DataDog/apm-python
.riot/requirements/24186ad.txt                                          @DataDog/apm-python
.riot/requirements/33e1d73.txt                                          @DataDog/apm-python
.riot/requirements/396b468.txt                                          @DataDog/apm-python
.riot/requirements/6b1a327.txt                                          @DataDog/apm-python
.riot/requirements/70a7600.txt                                          @DataDog/apm-python
.riot/requirements/a2f4af0.txt                                          @DataDog/apm-python
.riot/requirements/a8701c5.txt                                          @DataDog/apm-python
.riot/requirements/aa913fb.txt                                          @DataDog/apm-python
.riot/requirements/afcaf52.txt                                          @DataDog/apm-python
.riot/requirements/b1b5bdd.txt                                          @DataDog/apm-python
.riot/requirements/c2420c2.txt                                          @DataDog/apm-python
.riot/requirements/f47fe53.txt                                          @DataDog/apm-python

github-actions[bot] avatar Jun 26 '25 09:06 github-actions[bot]

Bootstrap import analysis

Comparison of import times between this PR and base.

Summary

The average import time from this PR is: 268 ± 5 ms.

The average import time from base is: 270 ± 4 ms.

The import time difference between this PR and base is: -2.2 ± 0.2 ms.

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 1.929 ms (0.72%)
ddtrace.bootstrap.sitecustomize 1.261 ms (0.47%)
ddtrace.bootstrap.preload 1.261 ms (0.47%)
ddtrace.internal.remoteconfig.client 0.653 ms (0.24%)
ddtrace 0.669 ms (0.25%)
ddtrace.internal._unpatched 0.032 ms (0.01%)
json 0.032 ms (0.01%)
json.decoder 0.032 ms (0.01%)
re 0.032 ms (0.01%)
enum 0.032 ms (0.01%)
types 0.032 ms (0.01%)

github-actions[bot] avatar Jun 26 '25 10:06 github-actions[bot]

Benchmarks

Benchmark execution time: 2025-07-03 08:41:16

Comparing candidate commit 50a2db3d165b711849f31333a271eb4b3d9c579b in PR branch juanjux/riotize-integrations-django with baseline commit 25fb075cf5778b4d294f9f93b09a8936f83363a4 in branch main.

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

scenario:iastaspectsospath-ospathnormcase_aspect

  • 🟥 execution_time [+332.936ns; +424.353ns] or [+9.458%; +12.055%]

pr-commenter[bot] avatar Jun 26 '25 10:06 pr-commenter[bot]