Skip to content

fix: check git env vars before using the command on the source code integration #14613

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

Jump to bottom
Merged
merged 4 commits into from
Sep 18, 2025
Merged

fix: check git env vars before using the command on the source code integration #14613

merged 4 commits into from
Sep 18, 2025
+69 −1

Conversation

juanjux
Copy link
Collaborator

@juanjux juanjux commented Sep 17, 2025 •
edited by brettlangdon
Loading

The sourcecode integration was using git to retrieve the commit hash and repository url. However, we already have defined DD_GIT_COMMIT_HASH and DD_GIT_REPOSITORY_URL for these cases, so check these vars before.

Checklist

  • 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 juanjux self-assigned this Sep 17, 2025
@juanjux juanjux requested a review from a team as a code owner September 17, 2025 15:18
@juanjux
changelog
3915472 
Signed-off-by: Juanjo Alvarez <[email protected]>
@juanjux juanjux requested a review from a team as a code owner September 17, 2025 15:18
@juanjux juanjux requested a review from P403n1x87 September 17, 2025 15:18
@juanjux
fmt
f4f990d 
Signed-off-by: Juanjo Alvarez <[email protected]>
@juanjux juanjux requested a review from gnufede September 17, 2025 15:23
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 17, 2025 •
edited
Loading

CODEOWNERS have been resolved as:

releasenotes/notes/sourcecode-check-env-vars-cab3d9b6311c2695.yaml      @DataDog/apm-python
tests/sourcecode/test_source_code_env_vars.py                           @DataDog/apm-core-python
ddtrace/sourcecode/_utils.py                                            @DataDog/apm-core-python

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 17, 2025 •
edited
Loading

Bootstrap import analysis

Comparison of import times between this PR and base.

Summary

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

The average import time from base is: 275 ± 7 ms.

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

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 1.998 ms (0.73%)
ddtrace.bootstrap.sitecustomize 1.317 ms (0.48%)
ddtrace.bootstrap.preload 1.317 ms (0.48%)
ddtrace.internal.remoteconfig.client 0.653 ms (0.24%)
ddtrace 0.681 ms (0.25%)
ddtrace.internal._unpatched 0.030 ms (0.01%)
json 0.030 ms (0.01%)
json.decoder 0.030 ms (0.01%)
re 0.030 ms (0.01%)
enum 0.030 ms (0.01%)
types 0.030 ms (0.01%)

@brettlangdon brettlangdon enabled auto-merge (squash) September 17, 2025 18:20
@pr-commenter
Copy link

pr-commenter bot commented Sep 18, 2025

Performance SLOs

Comparing candidate juanjux/sourcecode-use-git-env-vars (bc12e11) with baseline main (33c9c44)

📈 Performance Regressions (2 suites)
📈 iast_aspects - 40/40

✅ re_expand_aspect

Time: ✅ 32.242µs (SLO: <40.000µs 📉 -19.4%) vs baseline: +1.2%

Memory: ✅ 37.179MB (SLO: <39.000MB -4.7%) vs baseline: +4.6%

✅ re_expand_noaspect

Time: ✅ 30.205µs (SLO: <40.000µs 📉 -24.5%) vs baseline: +5.5%

Memory: ✅ 36.903MB (SLO: <39.000MB -5.4%) vs baseline: +3.8%

✅ re_findall_aspect

Time: ✅ 2.944µs (SLO: <10.000µs 📉 -70.6%) vs baseline: +1.5%

Memory: ✅ 36.884MB (SLO: <39.000MB -5.4%) vs baseline: +3.8%

✅ re_findall_noaspect

Time: ✅ 1.410µs (SLO: <10.000µs 📉 -85.9%) vs baseline: -1.1%

Memory: ✅ 37.061MB (SLO: <39.000MB -5.0%) vs baseline: +4.1%

✅ re_finditer_aspect

Time: ✅ 4.506µs (SLO: <10.000µs 📉 -54.9%) vs baseline: +1.6%

Memory: ✅ 36.923MB (SLO: <39.000MB -5.3%) vs baseline: +3.8%

✅ re_finditer_noaspect

Time: ✅ 1.407µs (SLO: <10.000µs 📉 -85.9%) vs baseline: ~same

Memory: ✅ 37.080MB (SLO: <39.000MB -4.9%) vs baseline: +4.2%

✅ re_fullmatch_aspect

Time: ✅ 2.846µs (SLO: <10.000µs 📉 -71.5%) vs baseline: +7.4%

Memory: ✅ 36.884MB (SLO: <39.000MB -5.4%) vs baseline: +3.9%

✅ re_fullmatch_noaspect

Time: ✅ 1.289µs (SLO: <10.000µs 📉 -87.1%) vs baseline: ~same

Memory: ✅ 37.041MB (SLO: <39.000MB -5.0%) vs baseline: +4.2%

✅ re_group_aspect

Time: ✅ 2.924µs (SLO: <10.000µs 📉 -70.8%) vs baseline: -0.1%

Memory: ✅ 37.139MB (SLO: <39.000MB -4.8%) vs baseline: +4.5%

✅ re_group_noaspect

Time: ✅ 1.600µs (SLO: <10.000µs 📉 -84.0%) vs baseline: +1.0%

Memory: ✅ 36.864MB (SLO: <39.000MB -5.5%) vs baseline: +3.7%

✅ re_groups_aspect

Time: ✅ 3.384µs (SLO: <10.000µs 📉 -66.2%) vs baseline: 📈 +10.4%

Memory: ✅ 37.159MB (SLO: <39.000MB -4.7%) vs baseline: +4.5%

✅ re_groups_noaspect

Time: ✅ 1.700µs (SLO: <10.000µs 📉 -83.0%) vs baseline: -1.0%

Memory: ✅ 36.943MB (SLO: <39.000MB -5.3%) vs baseline: +3.9%

✅ re_match_aspect

Time: ✅ 2.742µs (SLO: <10.000µs 📉 -72.6%) vs baseline: +1.3%

Memory: ✅ 37.120MB (SLO: <39.000MB -4.8%) vs baseline: +4.4%

✅ re_match_noaspect

Time: ✅ 1.289µs (SLO: <10.000µs 📉 -87.1%) vs baseline: +0.3%

Memory: ✅ 37.100MB (SLO: <39.000MB -4.9%) vs baseline: +4.4%

✅ re_search_aspect

Time: ✅ 2.558µs (SLO: <10.000µs 📉 -74.4%) vs baseline: ~same

Memory: ✅ 36.982MB (SLO: <39.000MB -5.2%) vs baseline: +4.0%

✅ re_search_noaspect

Time: ✅ 1.200µs (SLO: <10.000µs 📉 -88.0%) vs baseline: -0.1%

Memory: ✅ 37.041MB (SLO: <39.000MB -5.0%) vs baseline: +4.1%

✅ re_sub_aspect

Time: ✅ 3.435µs (SLO: <10.000µs 📉 -65.7%) vs baseline: -0.6%

Memory: ✅ 36.962MB (SLO: <39.000MB -5.2%) vs baseline: +4.1%

✅ re_sub_noaspect

Time: ✅ 1.535µs (SLO: <10.000µs 📉 -84.7%) vs baseline: +0.7%

Memory: ✅ 37.139MB (SLO: <39.000MB -4.8%) vs baseline: +4.6%

✅ re_subn_aspect

Time: ✅ 3.934µs (SLO: <10.000µs 📉 -60.7%) vs baseline: +6.6%

Memory: ✅ 37.120MB (SLO: <39.000MB -4.8%) vs baseline: +4.4%

✅ re_subn_noaspect

Time: ✅ 1.622µs (SLO: <10.000µs 📉 -83.8%) vs baseline: +1.8%

Memory: ✅ 36.903MB (SLO: <39.000MB -5.4%) vs baseline: +3.8%

📈 iastaspects - 118/118

✅ add_aspect

Time: ✅ 0.402µs (SLO: <10.000µs 📉 -96.0%) vs baseline: ~same

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

✅ add_inplace_aspect

Time: ✅ 0.406µs (SLO: <10.000µs 📉 -95.9%) vs baseline: -0.2%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ add_inplace_noaspect

Time: ✅ 0.324µs (SLO: <10.000µs 📉 -96.8%) vs baseline: +1.1%

Memory: ✅ 37.218MB (SLO: <39.000MB -4.6%) vs baseline: +4.6%

✅ add_noaspect

Time: ✅ 0.278µs (SLO: <10.000µs 📉 -97.2%) vs baseline: +0.6%

Memory: ✅ 37.336MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ bytearray_aspect

Time: ✅ 1.312µs (SLO: <10.000µs 📉 -86.9%) vs baseline: -0.7%

Memory: ✅ 37.336MB (SLO: <39.000MB -4.3%) vs baseline: +5.1%

✅ bytearray_extend_aspect

Time: ✅ 1.460µs (SLO: <10.000µs 📉 -85.4%) vs baseline: -0.8%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ bytearray_extend_noaspect

Time: ✅ 0.611µs (SLO: <10.000µs 📉 -93.9%) vs baseline: ~same

Memory: ✅ 37.218MB (SLO: <39.000MB -4.6%) vs baseline: +4.7%

✅ bytearray_noaspect

Time: ✅ 0.485µs (SLO: <10.000µs 📉 -95.1%) vs baseline: +1.1%

Memory: ✅ 37.336MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ bytes_aspect

Time: ✅ 1.522µs (SLO: <10.000µs 📉 -84.8%) vs baseline: 📈 +17.3%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.8%

✅ bytes_noaspect

Time: ✅ 0.492µs (SLO: <10.000µs 📉 -95.1%) vs baseline: -0.4%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ bytesio_aspect

Time: ✅ 1.347µs (SLO: <10.000µs 📉 -86.5%) vs baseline: +0.6%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ bytesio_noaspect

Time: ✅ 0.497µs (SLO: <10.000µs 📉 -95.0%) vs baseline: +0.2%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +4.9%

✅ capitalize_aspect

Time: ✅ 0.747µs (SLO: <10.000µs 📉 -92.5%) vs baseline: +2.0%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ capitalize_noaspect

Time: ✅ 0.438µs (SLO: <10.000µs 📉 -95.6%) vs baseline: +1.4%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.8%

✅ casefold_aspect

Time: ✅ 0.736µs (SLO: <10.000µs 📉 -92.6%) vs baseline: -0.6%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ casefold_noaspect

Time: ✅ 0.371µs (SLO: <10.000µs 📉 -96.3%) vs baseline: +0.5%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ decode_aspect

Time: ✅ 0.729µs (SLO: <10.000µs 📉 -92.7%) vs baseline: +0.7%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +4.8%

✅ decode_noaspect

Time: ✅ 0.419µs (SLO: <10.000µs 📉 -95.8%) vs baseline: +0.2%

Memory: ✅ 37.159MB (SLO: <39.000MB -4.7%) vs baseline: +4.6%

✅ encode_aspect

Time: ✅ 0.706µs (SLO: <10.000µs 📉 -92.9%) vs baseline: -1.3%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.1%

✅ encode_noaspect

Time: ✅ 0.401µs (SLO: <10.000µs 📉 -96.0%) vs baseline: +0.4%

Memory: ✅ 37.179MB (SLO: <39.000MB -4.7%) vs baseline: +4.9%

✅ format_aspect

Time: ✅ 3.395µs (SLO: <10.000µs 📉 -66.0%) vs baseline: +0.7%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ format_map_aspect

Time: ✅ 3.530µs (SLO: <10.000µs 📉 -64.7%) vs baseline: -0.4%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +5.0%

✅ format_map_noaspect

Time: ✅ 0.775µs (SLO: <10.000µs 📉 -92.2%) vs baseline: +0.3%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.8%

✅ format_noaspect

Time: ✅ 0.599µs (SLO: <10.000µs 📉 -94.0%) vs baseline: +0.3%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +4.9%

✅ index_aspect

Time: ✅ 0.357µs (SLO: <10.000µs 📉 -96.4%) vs baseline: +0.4%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ index_noaspect

Time: ✅ 0.275µs (SLO: <10.000µs 📉 -97.3%) vs baseline: +0.2%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.1%

✅ join_aspect

Time: ✅ 1.332µs (SLO: <10.000µs 📉 -86.7%) vs baseline: +0.6%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.8%

✅ join_noaspect

Time: ✅ 0.492µs (SLO: <10.000µs 📉 -95.1%) vs baseline: ~same

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

✅ ljust_aspect

Time: ✅ 2.551µs (SLO: <20.000µs 📉 -87.2%) vs baseline: -0.7%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +5.0%

✅ ljust_noaspect

Time: ✅ 0.405µs (SLO: <10.000µs 📉 -95.9%) vs baseline: -0.7%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +5.1%

✅ lower_aspect

Time: ✅ 2.274µs (SLO: <10.000µs 📉 -77.3%) vs baseline: -0.5%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

✅ lower_noaspect

Time: ✅ 0.371µs (SLO: <10.000µs 📉 -96.3%) vs baseline: +0.5%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

✅ lstrip_aspect

Time: ✅ 2.238µs (SLO: <20.000µs 📉 -88.8%) vs baseline: ~same

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.8%

✅ lstrip_noaspect

Time: ✅ 0.381µs (SLO: <10.000µs 📉 -96.2%) vs baseline: -1.7%

Memory: ✅ 37.336MB (SLO: <39.000MB -4.3%) vs baseline: +5.2%

✅ modulo_aspect

Time: ✅ 0.993µs (SLO: <10.000µs 📉 -90.1%) vs baseline: -0.5%

Memory: ✅ 37.218MB (SLO: <39.000MB -4.6%) vs baseline: +4.7%

✅ modulo_aspect_for_bytearray_bytearray

Time: ✅ 1.530µs (SLO: <10.000µs 📉 -84.7%) vs baseline: -0.1%

Memory: ✅ 37.336MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ modulo_aspect_for_bytes

Time: ✅ 0.970µs (SLO: <10.000µs 📉 -90.3%) vs baseline: +0.1%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ modulo_aspect_for_bytes_bytearray

Time: ✅ 1.195µs (SLO: <10.000µs 📉 -88.1%) vs baseline: +0.3%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.6%

✅ modulo_noaspect

Time: ✅ 0.629µs (SLO: <10.000µs 📉 -93.7%) vs baseline: +0.2%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +5.0%

✅ replace_aspect

Time: ✅ 5.695µs (SLO: <10.000µs 📉 -43.0%) vs baseline: 📈 +15.2%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

✅ replace_noaspect

Time: ✅ 0.463µs (SLO: <10.000µs 📉 -95.4%) vs baseline: +1.3%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.1%

✅ repr_aspect

Time: ✅ 0.901µs (SLO: <10.000µs 📉 -91.0%) vs baseline: -0.2%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +5.0%

✅ repr_noaspect

Time: ✅ 0.416µs (SLO: <10.000µs 📉 -95.8%) vs baseline: ~same

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.7%

✅ rstrip_aspect

Time: ✅ 1.947µs (SLO: <20.000µs 📉 -90.3%) vs baseline: +0.8%

Memory: ✅ 37.336MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ rstrip_noaspect

Time: ✅ 0.381µs (SLO: <10.000µs 📉 -96.2%) vs baseline: +0.6%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ slice_aspect

Time: ✅ 0.496µs (SLO: <10.000µs 📉 -95.0%) vs baseline: +0.7%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +5.0%

✅ slice_noaspect

Time: ✅ 0.448µs (SLO: <10.000µs 📉 -95.5%) vs baseline: -0.5%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ stringio_aspect

Time: ✅ 1.551µs (SLO: <10.000µs 📉 -84.5%) vs baseline: +0.4%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.1%

✅ stringio_noaspect

Time: ✅ 0.719µs (SLO: <10.000µs 📉 -92.8%) vs baseline: -0.1%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ strip_aspect

Time: ✅ 2.259µs (SLO: <20.000µs 📉 -88.7%) vs baseline: +0.6%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.8%

✅ strip_noaspect

Time: ✅ 0.388µs (SLO: <10.000µs 📉 -96.1%) vs baseline: +0.8%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +5.0%

✅ swapcase_aspect

Time: ✅ 2.493µs (SLO: <10.000µs 📉 -75.1%) vs baseline: ~same

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.7%

✅ swapcase_noaspect

Time: ✅ 0.539µs (SLO: <10.000µs 📉 -94.6%) vs baseline: +0.4%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ title_aspect

Time: ✅ 2.385µs (SLO: <10.000µs 📉 -76.1%) vs baseline: -1.7%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ title_noaspect

Time: ✅ 0.505µs (SLO: <10.000µs 📉 -95.0%) vs baseline: +0.5%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.7%

✅ translate_aspect

Time: ✅ 3.702µs (SLO: <10.000µs 📉 -63.0%) vs baseline: +9.7%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ translate_noaspect

Time: ✅ 1.040µs (SLO: <10.000µs 📉 -89.6%) vs baseline: ~same

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

✅ upper_aspect

Time: ✅ 2.522µs (SLO: <10.000µs 📉 -74.8%) vs baseline: 📈 +11.0%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ upper_noaspect

Time: ✅ 0.371µs (SLO: <10.000µs 📉 -96.3%) vs baseline: -0.3%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

🟡 Near SLO Breach (5 suites)
🟡 djangosimple - 26/26

✅ appsec

Time: ✅ 20.447ms (SLO: <22.300ms -8.3%) vs baseline: -0.5%

Memory: ✅ 64.409MB (SLO: <66.000MB -2.4%) vs baseline: +4.7%

✅ exception-replay-enabled

Time: ✅ 1.347ms (SLO: <1.450ms -7.1%) vs baseline: +0.3%

Memory: ✅ 63.486MB (SLO: <66.000MB -3.8%) vs baseline: +4.8%

✅ iast

Time: ✅ 20.533ms (SLO: <22.250ms -7.7%) vs baseline: +0.2%

Memory: ✅ 64.448MB (SLO: <66.000MB -2.4%) vs baseline: +4.8%

✅ profiler

Time: ✅ 15.228ms (SLO: <16.550ms -8.0%) vs baseline: -0.6%

Memory: ✅ 52.809MB (SLO: <53.500MB 🟡 -1.3%) vs baseline: +4.8%

✅ span-code-origin

Time: ✅ 26.088ms (SLO: <28.200ms -7.5%) vs baseline: -0.2%

Memory: ✅ 66.537MB (SLO: <68.500MB -2.9%) vs baseline: +4.9%

✅ tracer

Time: ✅ 20.517ms (SLO: <21.750ms -5.7%) vs baseline: ~same

Memory: ✅ 64.468MB (SLO: <66.000MB -2.3%) vs baseline: +5.0%

✅ tracer-and-profiler

Time: ✅ 22.074ms (SLO: <23.500ms -6.1%) vs baseline: ~same

Memory: ✅ 65.706MB (SLO: <67.000MB 🟡 -1.9%) vs baseline: +4.9%

✅ tracer-dont-create-db-spans

Time: ✅ 19.290ms (SLO: <21.500ms 📉 -10.3%) vs baseline: -0.2%

Memory: ✅ 64.448MB (SLO: <66.000MB -2.4%) vs baseline: +5.0%

✅ tracer-native

Time: ✅ 20.443ms (SLO: <21.750ms -6.0%) vs baseline: -0.2%

Memory: ✅ 65.647MB (SLO: <66.000MB 🟡 -0.5%) vs baseline: +4.8%

✅ tracer-no-caches

Time: ✅ 18.453ms (SLO: <19.650ms -6.1%) vs baseline: ~same

Memory: ✅ 64.428MB (SLO: <66.000MB -2.4%) vs baseline: +4.8%

✅ tracer-no-databases

Time: ✅ 18.673ms (SLO: <20.100ms -7.1%) vs baseline: -0.9%

Memory: ✅ 64.428MB (SLO: <66.000MB -2.4%) vs baseline: +4.9%

✅ tracer-no-middleware

Time: ✅ 20.159ms (SLO: <21.500ms -6.2%) vs baseline: -0.3%

Memory: ✅ 64.428MB (SLO: <66.000MB -2.4%) vs baseline: +4.8%

✅ tracer-no-templates

Time: ✅ 20.271ms (SLO: <22.000ms -7.9%) vs baseline: ~same

Memory: ✅ 64.350MB (SLO: <66.000MB -2.5%) vs baseline: +4.8%

🟡 errortrackingdjangosimple - 6/6

✅ errortracking-enabled-all

Time: ✅ 18.103ms (SLO: <19.850ms -8.8%) vs baseline: -0.2%

Memory: ✅ 64.428MB (SLO: <65.500MB 🟡 -1.6%) vs baseline: +4.8%

✅ errortracking-enabled-user

Time: ✅ 18.023ms (SLO: <19.400ms -7.1%) vs baseline: -0.3%

Memory: ✅ 64.408MB (SLO: <65.500MB 🟡 -1.7%) vs baseline: +4.9%

✅ tracer-enabled

Time: ✅ 18.126ms (SLO: <19.450ms -6.8%) vs baseline: +0.1%

Memory: ✅ 64.344MB (SLO: <65.500MB 🟡 -1.8%) vs baseline: +4.8%

🟡 flasksimple - 17/17

✅ appsec-get

Time: ✅ 4.580ms (SLO: <4.750ms -3.6%) vs baseline: +0.2%

Memory: ✅ 62.384MB (SLO: <64.500MB -3.3%) vs baseline: +4.9%

✅ appsec-post

Time: ✅ 6.569ms (SLO: <6.750ms -2.7%) vs baseline: -0.1%

Memory: ✅ 62.285MB (SLO: <64.500MB -3.4%) vs baseline: +4.8%

✅ appsec-telemetry

Time: ✅ 4.582ms (SLO: <4.750ms -3.5%) vs baseline: +0.4%

Memory: ✅ 62.285MB (SLO: <64.500MB -3.4%) vs baseline: +4.9%

✅ debugger

Time: ✅ 1.859ms (SLO: <2.000ms -7.0%) vs baseline: +0.2%

Memory: ✅ 44.728MB (SLO: <45.000MB 🟡 -0.6%) vs baseline: +4.9%

✅ iast-get

Time: ✅ 1.857ms (SLO: <2.000ms -7.1%) vs baseline: -0.4%

Memory: ✅ 41.622MB (SLO: <49.000MB 📉 -15.1%) vs baseline: +5.0%

✅ profiler

Time: ✅ 1.913ms (SLO: <2.100ms -8.9%) vs baseline: -0.4%

Memory: ✅ 44.374MB (SLO: <46.500MB -4.6%) vs baseline: +4.9%

✅ tracer

Time: ✅ 3.376ms (SLO: <3.650ms -7.5%) vs baseline: -0.2%

Memory: ✅ 51.433MB (SLO: <53.500MB -3.9%) vs baseline: +4.8%

✅ tracer-native

Time: ✅ 3.368ms (SLO: <3.650ms -7.7%) vs baseline: ~same

Memory: ✅ 52.691MB (SLO: <53.500MB 🟡 -1.5%) vs baseline: +5.0%

🟡 flasksqli - 6/6

✅ appsec-enabled

Time: ✅ 3.950ms (SLO: <4.200ms -5.9%) vs baseline: +0.5%

Memory: ✅ 62.489MB (SLO: <66.000MB -5.3%) vs baseline: +4.7%

✅ iast-enabled

Time: ✅ 2.467ms (SLO: <2.800ms 📉 -11.9%) vs baseline: +0.5%

Memory: ✅ 58.046MB (SLO: <59.000MB 🟡 -1.6%) vs baseline: +4.8%

✅ tracer-enabled

Time: ✅ 2.082ms (SLO: <2.250ms -7.5%) vs baseline: +0.1%

Memory: ✅ 51.208MB (SLO: <53.500MB -4.3%) vs baseline: +4.9%

🟡 otelspan - 22/22

✅ add-event

Time: ✅ 45.255ms (SLO: <47.150ms -4.0%) vs baseline: +0.6%

Memory: ✅ 44.469MB (SLO: <46.500MB -4.4%) vs baseline: +4.9%

✅ add-metrics

Time: ✅ 321.395ms (SLO: <344.800ms -6.8%) vs baseline: +0.6%

Memory: ✅ 554.820MB (SLO: <562.000MB 🟡 -1.3%) vs baseline: +5.1%

✅ add-tags

Time: ✅ 293.509ms (SLO: <314.000ms -6.5%) vs baseline: -0.2%

Memory: ✅ 555.561MB (SLO: <563.500MB 🟡 -1.4%) vs baseline: +4.9%

✅ get-context

Time: ✅ 84.585ms (SLO: <92.350ms -8.4%) vs baseline: +2.2%

Memory: ✅ 39.677MB (SLO: <46.500MB 📉 -14.7%) vs baseline: +4.8%

✅ is-recording

Time: ✅ 42.923ms (SLO: <44.500ms -3.5%) vs baseline: -0.2%

Memory: ✅ 43.889MB (SLO: <46.500MB -5.6%) vs baseline: +4.8%

✅ record-exception

Time: ✅ 61.677ms (SLO: <67.650ms -8.8%) vs baseline: -0.3%

Memory: ✅ 39.951MB (SLO: <46.500MB 📉 -14.1%) vs baseline: +5.0%

✅ set-status

Time: ✅ 49.560ms (SLO: <50.400ms 🟡 -1.7%) vs baseline: +1.5%

Memory: ✅ 43.902MB (SLO: <46.500MB -5.6%) vs baseline: +4.9%

✅ start

Time: ✅ 42.264ms (SLO: <43.450ms -2.7%) vs baseline: +0.1%

Memory: ✅ 43.888MB (SLO: <46.500MB -5.6%) vs baseline: +4.9%

✅ start-finish

Time: ✅ 83.179ms (SLO: <88.000ms -5.5%) vs baseline: +0.3%

Memory: ✅ 33.777MB (SLO: <46.500MB 📉 -27.4%) vs baseline: +4.5%

✅ start-finish-telemetry

Time: ✅ 84.447ms (SLO: <89.000ms -5.1%) vs baseline: -0.3%

Memory: ✅ 33.895MB (SLO: <46.500MB 📉 -27.1%) vs baseline: +5.2%

✅ update-name

Time: ✅ 44.060ms (SLO: <45.150ms -2.4%) vs baseline: -0.4%

Memory: ✅ 44.232MB (SLO: <46.500MB -4.9%) vs baseline: +5.0%

⚠️ Unstable Tests (1 suite)
⚠️ coreapiscenario - 12/12 (2 unstable)

⚠️ context_with_data_listeners

Time: ⚠️ 13.435µs (SLO: <20.000µs 📉 -32.8%) vs baseline: +0.4%

Memory: ✅ 31.300MB (SLO: <33.500MB -6.6%) vs baseline: +4.9%

✅ context_with_data_no_listeners

Time: ✅ 3.379µs (SLO: <10.000µs 📉 -66.2%) vs baseline: -0.3%

Memory: ✅ 31.261MB (SLO: <33.500MB -6.7%) vs baseline: +4.7%

⚠️ context_with_data_only_all_listeners

Time: ⚠️ 13.335µs (SLO: <20.000µs 📉 -33.3%) vs baseline: -0.8%

Memory: ✅ 31.339MB (SLO: <33.500MB -6.4%) vs baseline: +4.9%

✅ get_item_exists

Time: ✅ 0.583µs (SLO: <10.000µs 📉 -94.2%) vs baseline: -0.7%

Memory: ✅ 31.339MB (SLO: <33.500MB -6.4%) vs baseline: +4.9%

✅ get_item_missing

Time: ✅ 0.635µs (SLO: <10.000µs 📉 -93.6%) vs baseline: -0.4%

Memory: ✅ 31.280MB (SLO: <33.500MB -6.6%) vs baseline: +4.8%

✅ set_item

Time: ✅ 24.270µs (SLO: <30.000µs 📉 -19.1%) vs baseline: -0.6%

Memory: ✅ 31.339MB (SLO: <33.500MB -6.4%) vs baseline: +5.0%

✅ All Tests Passing (16 suites)
✅ errortrackingflasksqli - 6/6

✅ errortracking-enabled-all

Time: ✅ 2.091ms (SLO: <2.300ms -9.1%) vs baseline: -0.2%

Memory: ✅ 51.177MB (SLO: <53.500MB -4.3%) vs baseline: +4.8%

✅ errortracking-enabled-user

Time: ✅ 2.092ms (SLO: <2.250ms -7.0%) vs baseline: -0.3%

Memory: ✅ 51.135MB (SLO: <53.500MB -4.4%) vs baseline: +4.7%

✅ tracer-enabled

Time: ✅ 2.093ms (SLO: <2.300ms -9.0%) vs baseline: -0.1%

Memory: ✅ 51.175MB (SLO: <53.500MB -4.3%) vs baseline: +4.9%

✅ httppropagationextract - 60/60

✅ all_styles_all_headers

Time: ✅ 80.806µs (SLO: <100.000µs 📉 -19.2%) vs baseline: -0.2%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +5.1%

✅ b3_headers

Time: ✅ 14.283µs (SLO: <20.000µs 📉 -28.6%) vs baseline: +0.4%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ b3_single_headers

Time: ✅ 13.360µs (SLO: <20.000µs 📉 -33.2%) vs baseline: +0.4%

Memory: ✅ 31.438MB (SLO: <33.500MB -6.2%) vs baseline: +4.8%

✅ datadog_tracecontext_tracestate_not_propagated_on_trace_id_no_match

Time: ✅ 64.745µs (SLO: <80.000µs 📉 -19.1%) vs baseline: +2.1%

Memory: ✅ 31.339MB (SLO: <33.500MB -6.4%) vs baseline: +4.5%

✅ datadog_tracecontext_tracestate_propagated_on_trace_id_match

Time: ✅ 65.933µs (SLO: <80.000µs 📉 -17.6%) vs baseline: -0.2%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ empty_headers

Time: ✅ 1.619µs (SLO: <10.000µs 📉 -83.8%) vs baseline: +0.6%

Memory: ✅ 31.379MB (SLO: <33.500MB -6.3%) vs baseline: +4.7%

✅ full_t_id_datadog_headers

Time: ✅ 22.873µs (SLO: <30.000µs 📉 -23.8%) vs baseline: ~same

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.1%

✅ invalid_priority_header

Time: ✅ 6.513µs (SLO: <10.000µs 📉 -34.9%) vs baseline: -0.4%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ invalid_span_id_header

Time: ✅ 6.511µs (SLO: <10.000µs 📉 -34.9%) vs baseline: +0.4%

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.3%

✅ invalid_tags_header

Time: ✅ 6.543µs (SLO: <10.000µs 📉 -34.6%) vs baseline: ~same

Memory: ✅ 31.359MB (SLO: <33.500MB -6.4%) vs baseline: +4.6%

✅ invalid_trace_id_header

Time: ✅ 6.541µs (SLO: <10.000µs 📉 -34.6%) vs baseline: ~same

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +4.9%

✅ large_header_no_matches

Time: ✅ 27.470µs (SLO: <30.000µs -8.4%) vs baseline: -0.7%

Memory: ✅ 31.438MB (SLO: <33.500MB -6.2%) vs baseline: +4.9%

✅ large_valid_headers_all

Time: ✅ 28.533µs (SLO: <40.000µs 📉 -28.7%) vs baseline: -0.3%

Memory: ✅ 31.398MB (SLO: <33.500MB -6.3%) vs baseline: +5.0%

✅ medium_header_no_matches

Time: ✅ 9.905µs (SLO: <20.000µs 📉 -50.5%) vs baseline: +1.0%

Memory: ✅ 31.438MB (SLO: <33.500MB -6.2%) vs baseline: +4.6%

✅ medium_valid_headers_all

Time: ✅ 11.258µs (SLO: <20.000µs 📉 -43.7%) vs baseline: +0.4%

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.0%

✅ none_propagation_style

Time: ✅ 1.691µs (SLO: <10.000µs 📉 -83.1%) vs baseline: -0.9%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ tracecontext_headers

Time: ✅ 34.777µs (SLO: <40.000µs 📉 -13.1%) vs baseline: +1.8%

Memory: ✅ 31.339MB (SLO: <33.500MB -6.4%) vs baseline: +4.6%

✅ valid_headers_all

Time: ✅ 6.501µs (SLO: <10.000µs 📉 -35.0%) vs baseline: ~same

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.0%

✅ valid_headers_basic

Time: ✅ 6.115µs (SLO: <10.000µs 📉 -38.8%) vs baseline: ~same

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ wsgi_empty_headers

Time: ✅ 1.592µs (SLO: <10.000µs 📉 -84.1%) vs baseline: -1.2%

Memory: ✅ 31.438MB (SLO: <33.500MB -6.2%) vs baseline: +5.0%

✅ wsgi_invalid_priority_header

Time: ✅ 6.653µs (SLO: <10.000µs 📉 -33.5%) vs baseline: +1.3%

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.1%

✅ wsgi_invalid_span_id_header

Time: ✅ 1.590µs (SLO: <10.000µs 📉 -84.1%) vs baseline: -0.6%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.9%

✅ wsgi_invalid_tags_header

Time: ✅ 6.533µs (SLO: <10.000µs 📉 -34.7%) vs baseline: -0.9%

Memory: ✅ 31.497MB (SLO: <33.500MB -6.0%) vs baseline: +5.2%

✅ wsgi_invalid_trace_id_header

Time: ✅ 6.562µs (SLO: <10.000µs 📉 -34.4%) vs baseline: +0.4%

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.1%

✅ wsgi_large_header_no_matches

Time: ✅ 28.750µs (SLO: <40.000µs 📉 -28.1%) vs baseline: +0.3%

Memory: ✅ 31.398MB (SLO: <33.500MB -6.3%) vs baseline: +4.8%

✅ wsgi_large_valid_headers_all

Time: ✅ 29.777µs (SLO: <40.000µs 📉 -25.6%) vs baseline: -0.2%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ wsgi_medium_header_no_matches

Time: ✅ 10.255µs (SLO: <20.000µs 📉 -48.7%) vs baseline: +2.1%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ wsgi_medium_valid_headers_all

Time: ✅ 11.570µs (SLO: <20.000µs 📉 -42.1%) vs baseline: +1.2%

Memory: ✅ 31.438MB (SLO: <33.500MB -6.2%) vs baseline: +5.0%

✅ wsgi_valid_headers_all

Time: ✅ 6.604µs (SLO: <10.000µs 📉 -34.0%) vs baseline: +0.2%

Memory: ✅ 31.438MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ wsgi_valid_headers_basic

Time: ✅ 6.058µs (SLO: <10.000µs 📉 -39.4%) vs baseline: -1.1%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ httppropagationinject - 16/16

✅ ids_only

Time: ✅ 21.925µs (SLO: <30.000µs 📉 -26.9%) vs baseline: ~same

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.1%

✅ with_all

Time: ✅ 29.116µs (SLO: <40.000µs 📉 -27.2%) vs baseline: -0.7%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.5%

✅ with_dd_origin

Time: ✅ 26.072µs (SLO: <30.000µs 📉 -13.1%) vs baseline: +1.2%

Memory: ✅ 31.438MB (SLO: <33.500MB -6.2%) vs baseline: +5.1%

✅ with_priority_and_origin

Time: ✅ 25.515µs (SLO: <40.000µs 📉 -36.2%) vs baseline: +2.1%

Memory: ✅ 31.379MB (SLO: <33.500MB -6.3%) vs baseline: +4.7%

✅ with_sampling_priority

Time: ✅ 21.773µs (SLO: <30.000µs 📉 -27.4%) vs baseline: +0.4%

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.1%

✅ with_tags

Time: ✅ 27.320µs (SLO: <40.000µs 📉 -31.7%) vs baseline: -0.3%

Memory: ✅ 31.398MB (SLO: <33.500MB -6.3%) vs baseline: +4.9%

✅ with_tags_invalid

Time: ✅ 29.888µs (SLO: <40.000µs 📉 -25.3%) vs baseline: +3.8%

Memory: ✅ 31.457MB (SLO: <33.500MB -6.1%) vs baseline: +5.1%

✅ with_tags_max_size

Time: ✅ 27.953µs (SLO: <40.000µs 📉 -30.1%) vs baseline: +0.9%

Memory: ✅ 31.418MB (SLO: <33.500MB -6.2%) vs baseline: +4.7%

✅ iastaspectsospath - 24/24

✅ ospathbasename_aspect

Time: ✅ 4.379µs (SLO: <10.000µs 📉 -56.2%) vs baseline: +0.3%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +5.0%

✅ ospathbasename_noaspect

Time: ✅ 1.069µs (SLO: <10.000µs 📉 -89.3%) vs baseline: -0.6%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.6%

✅ ospathjoin_aspect

Time: ✅ 6.377µs (SLO: <10.000µs 📉 -36.2%) vs baseline: +1.5%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.0%

✅ ospathjoin_noaspect

Time: ✅ 2.287µs (SLO: <10.000µs 📉 -77.1%) vs baseline: +0.2%

Memory: ✅ 37.218MB (SLO: <39.000MB -4.6%) vs baseline: +4.7%

✅ ospathnormcase_aspect

Time: ✅ 3.698µs (SLO: <10.000µs 📉 -63.0%) vs baseline: -1.1%

Memory: ✅ 37.257MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ ospathnormcase_noaspect

Time: ✅ 0.560µs (SLO: <10.000µs 📉 -94.4%) vs baseline: -1.2%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.7%

✅ ospathsplit_aspect

Time: ✅ 4.957µs (SLO: <10.000µs 📉 -50.4%) vs baseline: +0.4%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ ospathsplit_noaspect

Time: ✅ 1.580µs (SLO: <10.000µs 📉 -84.2%) vs baseline: -0.2%

Memory: ✅ 37.316MB (SLO: <39.000MB -4.3%) vs baseline: +5.1%

✅ ospathsplitdrive_aspect

Time: ✅ 3.942µs (SLO: <10.000µs 📉 -60.6%) vs baseline: +0.9%

Memory: ✅ 37.218MB (SLO: <39.000MB -4.6%) vs baseline: +4.6%

✅ ospathsplitdrive_noaspect

Time: ✅ 0.689µs (SLO: <10.000µs 📉 -93.1%) vs baseline: -0.8%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +5.0%

✅ ospathsplitext_aspect

Time: ✅ 4.684µs (SLO: <10.000µs 📉 -53.2%) vs baseline: +0.6%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ ospathsplitext_noaspect

Time: ✅ 1.376µs (SLO: <10.000µs 📉 -86.2%) vs baseline: -0.2%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.7%

✅ iastaspectssplit - 12/12

✅ rsplit_aspect

Time: ✅ 1.429µs (SLO: <10.000µs 📉 -85.7%) vs baseline: -0.2%

Memory: ✅ 37.336MB (SLO: <39.000MB -4.3%) vs baseline: +5.1%

✅ rsplit_noaspect

Time: ✅ 0.581µs (SLO: <10.000µs 📉 -94.2%) vs baseline: -0.1%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ split_aspect

Time: ✅ 1.424µs (SLO: <10.000µs 📉 -85.8%) vs baseline: +0.3%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ split_noaspect

Time: ✅ 0.571µs (SLO: <10.000µs 📉 -94.3%) vs baseline: -0.7%

Memory: ✅ 37.297MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ splitlines_aspect

Time: ✅ 1.389µs (SLO: <10.000µs 📉 -86.1%) vs baseline: +0.5%

Memory: ✅ 37.238MB (SLO: <39.000MB -4.5%) vs baseline: +4.9%

✅ splitlines_noaspect

Time: ✅ 0.586µs (SLO: <10.000µs 📉 -94.1%) vs baseline: +1.0%

Memory: ✅ 37.277MB (SLO: <39.000MB -4.4%) vs baseline: +4.9%

✅ iastpropagation - 8/8

✅ no-propagation

Time: ✅ 48.773µs (SLO: <60.000µs 📉 -18.7%) vs baseline: -0.3%

Memory: ✅ 36.962MB (SLO: <39.000MB -5.2%) vs baseline: +5.1%

✅ propagation_enabled

Time: ✅ 166.302µs (SLO: <190.000µs 📉 -12.5%) vs baseline: -0.2%

Memory: ✅ 36.903MB (SLO: <39.000MB -5.4%) vs baseline: +5.0%

✅ propagation_enabled_100

Time: ✅ 1.836ms (SLO: <1.900ms -3.3%) vs baseline: -0.8%

Memory: ✅ 36.884MB (SLO: <39.000MB -5.4%) vs baseline: +4.7%

✅ propagation_enabled_1000

Time: ✅ 31.792ms (SLO: <35.550ms 📉 -10.6%) vs baseline: -0.3%

Memory: ✅ 36.864MB (SLO: <39.000MB -5.5%) vs baseline: +4.8%

✅ otelsdkspan - 24/24

✅ add-event

Time: ✅ 40.470ms (SLO: <42.000ms -3.6%) vs baseline: +0.1%

Memory: ✅ 34.170MB (SLO: <39.000MB 📉 -12.4%) vs baseline: +5.0%

✅ add-link

Time: ✅ 36.271ms (SLO: <38.550ms -5.9%) vs baseline: -0.2%

Memory: ✅ 34.131MB (SLO: <39.000MB 📉 -12.5%) vs baseline: +4.8%

✅ add-metrics

Time: ✅ 219.745ms (SLO: <232.000ms -5.3%) vs baseline: -0.2%

Memory: ✅ 34.210MB (SLO: <39.000MB 📉 -12.3%) vs baseline: +4.9%

✅ add-tags

Time: ✅ 212.985ms (SLO: <221.600ms -3.9%) vs baseline: ~same

Memory: ✅ 34.151MB (SLO: <39.000MB 📉 -12.4%) vs baseline: +5.0%

✅ get-context

Time: ✅ 29.213ms (SLO: <31.300ms -6.7%) vs baseline: -0.2%

Memory: ✅ 34.190MB (SLO: <39.000MB 📉 -12.3%) vs baseline: +4.9%

✅ is-recording

Time: ✅ 29.094ms (SLO: <31.000ms -6.1%) vs baseline: +0.2%

Memory: ✅ 34.131MB (SLO: <39.000MB 📉 -12.5%) vs baseline: +4.8%

✅ record-exception

Time: ✅ 62.971ms (SLO: <65.850ms -4.4%) vs baseline: -0.7%

Memory: ✅ 34.170MB (SLO: <39.000MB 📉 -12.4%) vs baseline: +4.9%

✅ set-status

Time: ✅ 31.896ms (SLO: <34.150ms -6.6%) vs baseline: -0.6%

Memory: ✅ 34.151MB (SLO: <39.000MB 📉 -12.4%) vs baseline: +4.7%

✅ start

Time: ✅ 28.696ms (SLO: <30.150ms -4.8%) vs baseline: -0.9%

Memory: ✅ 34.151MB (SLO: <39.000MB 📉 -12.4%) vs baseline: +4.8%

✅ start-finish

Time: ✅ 33.749ms (SLO: <35.350ms -4.5%) vs baseline: -0.9%

Memory: ✅ 34.151MB (SLO: <39.000MB 📉 -12.4%) vs baseline: +4.9%

✅ start-finish-telemetry

Time: ✅ 33.634ms (SLO: <35.450ms -5.1%) vs baseline: -1.2%

Memory: ✅ 34.111MB (SLO: <39.000MB 📉 -12.5%) vs baseline: +4.8%

✅ update-name

Time: ✅ 30.718ms (SLO: <33.400ms -8.0%) vs baseline: -0.9%

Memory: ✅ 34.111MB (SLO: <39.000MB 📉 -12.5%) vs baseline: +4.8%

✅ packagespackageforrootmodulemapping - 4/4

✅ cache_off

Time: ✅ 338.011ms (SLO: <354.300ms -4.6%) vs baseline: -0.7%

Memory: ✅ 37.139MB (SLO: <40.000MB -7.2%) vs baseline: +4.6%

✅ cache_on

Time: ✅ 0.381µs (SLO: <10.000µs 📉 -96.2%) vs baseline: +0.6%

Memory: ✅ 36.132MB (SLO: <39.000MB -7.4%) vs baseline: +4.8%

✅ packagesupdateimporteddependencies - 24/24

✅ import_many

Time: ✅ 154.495µs (SLO: <170.000µs -9.1%) vs baseline: -0.8%

Memory: ✅ 36.052MB (SLO: <38.500MB -6.4%) vs baseline: +4.9%

✅ import_many_cached

Time: ✅ 120.732µs (SLO: <130.000µs -7.1%) vs baseline: +0.1%

Memory: ✅ 36.000MB (SLO: <38.500MB -6.5%) vs baseline: +4.8%

✅ import_many_stdlib

Time: ✅ 1.605ms (SLO: <1.750ms -8.3%) vs baseline: ~same

Memory: ✅ 36.103MB (SLO: <38.500MB -6.2%) vs baseline: +4.8%

✅ import_many_stdlib_cached

Time: ✅ 0.961ms (SLO: <1.100ms 📉 -12.6%) vs baseline: -0.2%

Memory: ✅ 36.136MB (SLO: <38.500MB -6.1%) vs baseline: +4.5%

✅ import_many_unknown

Time: ✅ 828.882µs (SLO: <890.000µs -6.9%) vs baseline: -0.7%

Memory: ✅ 36.241MB (SLO: <38.500MB -5.9%) vs baseline: +5.0%

✅ import_many_unknown_cached

Time: ✅ 793.325µs (SLO: <870.000µs -8.8%) vs baseline: +0.5%

Memory: ✅ 36.052MB (SLO: <38.500MB -6.4%) vs baseline: +4.5%

✅ import_one

Time: ✅ 19.824µs (SLO: <30.000µs 📉 -33.9%) vs baseline: ~same

Memory: ✅ 35.965MB (SLO: <39.000MB -7.8%) vs baseline: +4.6%

✅ import_one_cache

Time: ✅ 6.271µs (SLO: <10.000µs 📉 -37.3%) vs baseline: -0.4%

Memory: ✅ 36.191MB (SLO: <38.500MB -6.0%) vs baseline: +5.4%

✅ import_one_stdlib

Time: ✅ 18.652µs (SLO: <20.000µs -6.7%) vs baseline: ~same

Memory: ✅ 36.140MB (SLO: <38.500MB -6.1%) vs baseline: +5.3%

✅ import_one_stdlib_cache

Time: ✅ 6.286µs (SLO: <10.000µs 📉 -37.1%) vs baseline: +0.3%

Memory: ✅ 36.125MB (SLO: <38.500MB -6.2%) vs baseline: +5.4%

✅ import_one_unknown

Time: ✅ 45.398µs (SLO: <50.000µs -9.2%) vs baseline: +0.5%

Memory: ✅ 36.345MB (SLO: <38.500MB -5.6%) vs baseline: +5.8%

✅ import_one_unknown_cache

Time: ✅ 6.284µs (SLO: <10.000µs 📉 -37.2%) vs baseline: -0.1%

Memory: ✅ 36.067MB (SLO: <38.500MB -6.3%) vs baseline: +5.0%

✅ ratelimiter - 12/12

✅ defaults

Time: ✅ 2.340µs (SLO: <10.000µs 📉 -76.6%) vs baseline: -0.6%

Memory: ✅ 30.926MB (SLO: <34.000MB -9.0%) vs baseline: +5.1%

✅ high_rate_limit

Time: ✅ 2.429µs (SLO: <10.000µs 📉 -75.7%) vs baseline: +0.8%

Memory: ✅ 30.946MB (SLO: <34.000MB -9.0%) vs baseline: +4.7%

✅ long_window

Time: ✅ 2.347µs (SLO: <10.000µs 📉 -76.5%) vs baseline: -0.6%

Memory: ✅ 30.926MB (SLO: <34.000MB -9.0%) vs baseline: +4.8%

✅ low_rate_limit

Time: ✅ 2.345µs (SLO: <10.000µs 📉 -76.5%) vs baseline: -1.2%

Memory: ✅ 30.907MB (SLO: <34.000MB -9.1%) vs baseline: +4.7%

✅ no_rate_limit

Time: ✅ 0.828µs (SLO: <10.000µs 📉 -91.7%) vs baseline: -0.2%

Memory: ✅ 30.946MB (SLO: <34.000MB -9.0%) vs baseline: +4.9%

✅ short_window

Time: ✅ 2.482µs (SLO: <10.000µs 📉 -75.2%) vs baseline: -0.2%

Memory: ✅ 30.907MB (SLO: <34.000MB -9.1%) vs baseline: +4.7%

✅ recursivecomputation - 8/8

✅ deep

Time: ✅ 309.752ms (SLO: <320.950ms -3.5%) vs baseline: ~same

Memory: ✅ 32.480MB (SLO: <34.500MB -5.9%) vs baseline: +5.0%

✅ deep-profiled

Time: ✅ 328.138ms (SLO: <359.150ms -8.6%) vs baseline: +0.2%

Memory: ✅ 35.193MB (SLO: <39.000MB -9.8%) vs baseline: +4.9%

✅ medium

Time: ✅ 7.080ms (SLO: <7.400ms -4.3%) vs baseline: ~same

Memory: ✅ 31.221MB (SLO: <34.000MB -8.2%) vs baseline: +4.5%

✅ shallow

Time: ✅ 0.953ms (SLO: <1.050ms -9.2%) vs baseline: -0.3%

Memory: ✅ 31.280MB (SLO: <34.000MB -8.0%) vs baseline: +4.5%

✅ samplingrules - 8/8

✅ average_match

Time: ✅ 148.491µs (SLO: <290.000µs 📉 -48.8%) vs baseline: +0.5%

Memory: ✅ 31.320MB (SLO: <34.000MB -7.9%) vs baseline: +4.9%

✅ high_match

Time: ✅ 194.493µs (SLO: <480.000µs 📉 -59.5%) vs baseline: -0.2%

Memory: ✅ 31.339MB (SLO: <34.000MB -7.8%) vs baseline: +4.9%

✅ low_match

Time: ✅ 99.214µs (SLO: <120.000µs 📉 -17.3%) vs baseline: +0.2%

Memory: ✅ 631.914MB (SLO: <700.000MB -9.7%) vs baseline: +4.9%

✅ very_low_match

Time: ✅ 2.914ms (SLO: <8.500ms 📉 -65.7%) vs baseline: +0.5%

Memory: ✅ 69.587MB (SLO: <75.000MB -7.2%) vs baseline: +4.8%

✅ sethttpmeta - 32/32

✅ all-disabled

Time: ✅ 10.666µs (SLO: <20.000µs 📉 -46.7%) vs baseline: ~same

Memory: ✅ 31.811MB (SLO: <34.000MB -6.4%) vs baseline: +4.8%

✅ all-enabled

Time: ✅ 40.092µs (SLO: <50.000µs 📉 -19.8%) vs baseline: -0.4%

Memory: ✅ 31.831MB (SLO: <34.000MB -6.4%) vs baseline: +3.7%

✅ collectipvariant_exists

Time: ✅ 40.698µs (SLO: <50.000µs 📉 -18.6%) vs baseline: -0.2%

Memory: ✅ 31.831MB (SLO: <34.000MB -6.4%) vs baseline: +4.1%

✅ no-collectipvariant

Time: ✅ 39.967µs (SLO: <50.000µs 📉 -20.1%) vs baseline: ~same

Memory: ✅ 31.850MB (SLO: <34.000MB -6.3%) vs baseline: +4.9%

✅ no-useragentvariant

Time: ✅ 38.845µs (SLO: <50.000µs 📉 -22.3%) vs baseline: +0.3%

Memory: ✅ 32.086MB (SLO: <34.000MB -5.6%) vs baseline: +5.6%

✅ obfuscation-no-query

Time: ✅ 40.492µs (SLO: <50.000µs 📉 -19.0%) vs baseline: -0.6%

Memory: ✅ 31.850MB (SLO: <34.000MB -6.3%) vs baseline: +4.8%

✅ obfuscation-regular-case-explicit-query

Time: ✅ 75.868µs (SLO: <90.000µs 📉 -15.7%) vs baseline: +0.1%

Memory: ✅ 32.244MB (SLO: <34.000MB -5.2%) vs baseline: +5.0%

✅ obfuscation-regular-case-implicit-query

Time: ✅ 76.441µs (SLO: <90.000µs 📉 -15.1%) vs baseline: ~same

Memory: ✅ 32.224MB (SLO: <34.000MB -5.2%) vs baseline: +4.9%

✅ obfuscation-send-querystring-disabled

Time: ✅ 153.722µs (SLO: <170.000µs -9.6%) vs baseline: ~same

Memory: ✅ 32.204MB (SLO: <34.500MB -6.7%) vs baseline: +4.9%

✅ obfuscation-worst-case-explicit-query

Time: ✅ 148.193µs (SLO: <160.000µs -7.4%) vs baseline: ~same

Memory: ✅ 32.204MB (SLO: <34.500MB -6.7%) vs baseline: +4.7%

✅ obfuscation-worst-case-implicit-query

Time: ✅ 154.371µs (SLO: <170.000µs -9.2%) vs baseline: ~same

Memory: ✅ 32.224MB (SLO: <34.500MB -6.6%) vs baseline: +4.9%

✅ useragentvariant_exists_1

Time: ✅ 39.584µs (SLO: <50.000µs 📉 -20.8%) vs baseline: +0.5%

Memory: ✅ 31.870MB (SLO: <34.000MB -6.3%) vs baseline: +4.1%

✅ useragentvariant_exists_2

Time: ✅ 40.578µs (SLO: <50.000µs 📉 -18.8%) vs baseline: -0.3%

Memory: ✅ 31.831MB (SLO: <34.000MB -6.4%) vs baseline: +4.1%

✅ useragentvariant_exists_3

Time: ✅ 40.324µs (SLO: <50.000µs 📉 -19.4%) vs baseline: ~same

Memory: ✅ 31.811MB (SLO: <34.000MB -6.4%) vs baseline: +4.0%

✅ useragentvariant_not_exists_1

Time: ✅ 39.578µs (SLO: <50.000µs 📉 -20.8%) vs baseline: +0.3%

Memory: ✅ 31.831MB (SLO: <34.000MB -6.4%) vs baseline: +3.7%

✅ useragentvariant_not_exists_2

Time: ✅ 40.021µs (SLO: <50.000µs 📉 -20.0%) vs baseline: +1.5%

Memory: ✅ 31.850MB (SLO: <34.000MB -6.3%) vs baseline: +4.3%

✅ span - 26/26

✅ add-event

Time: ✅ 24.410ms (SLO: <26.200ms -6.8%) vs baseline: +1.0%

Memory: ✅ 50.989MB (SLO: <53.000MB -3.8%) vs baseline: +4.9%

✅ add-metrics

Time: ✅ 92.460ms (SLO: <98.350ms -6.0%) vs baseline: -0.1%

Memory: ✅ 607.190MB (SLO: <961.000MB 📉 -36.8%) vs baseline: +5.0%

✅ add-tags

Time: ✅ 150.838ms (SLO: <168.550ms 📉 -10.5%) vs baseline: +0.4%

Memory: ✅ 607.515MB (SLO: <962.500MB 📉 -36.9%) vs baseline: +4.9%

✅ get-context

Time: ✅ 22.602ms (SLO: <23.700ms -4.6%) vs baseline: +1.4%

Memory: ✅ 49.749MB (SLO: <53.000MB -6.1%) vs baseline: +4.8%

✅ is-recording

Time: ✅ 22.780ms (SLO: <23.900ms -4.7%) vs baseline: +1.8%

Memory: ✅ 49.814MB (SLO: <53.000MB -6.0%) vs baseline: +5.0%

✅ record-exception

Time: ✅ 42.613ms (SLO: <44.500ms -4.2%) vs baseline: +0.7%

Memory: ✅ 42.674MB (SLO: <53.000MB 📉 -19.5%) vs baseline: +5.2%

✅ set-status

Time: ✅ 24.700ms (SLO: <26.000ms -5.0%) vs baseline: +1.5%

Memory: ✅ 49.818MB (SLO: <53.000MB -6.0%) vs baseline: +5.0%

✅ start

Time: ✅ 22.277ms (SLO: <23.500ms -5.2%) vs baseline: +0.7%

Memory: ✅ 49.722MB (SLO: <53.000MB -6.2%) vs baseline: +4.7%

✅ start-finish

Time: ✅ 52.146ms (SLO: <55.500ms -6.0%) vs baseline: ~same

Memory: ✅ 31.339MB (SLO: <34.000MB -7.8%) vs baseline: +5.1%

✅ start-finish-telemetry

Time: ✅ 53.504ms (SLO: <58.300ms -8.2%) vs baseline: -0.2%

Memory: ✅ 31.398MB (SLO: <34.000MB -7.7%) vs baseline: +5.1%

✅ start-finish-traceid128

Time: ✅ 55.879ms (SLO: <60.050ms -6.9%) vs baseline: +0.6%

Memory: ✅ 31.359MB (SLO: <34.000MB -7.8%) vs baseline: +5.0%

✅ start-traceid128

Time: ✅ 22.482ms (SLO: <24.600ms -8.6%) vs baseline: -0.7%

Memory: ✅ 49.751MB (SLO: <53.000MB -6.1%) vs baseline: +4.7%

✅ update-name

Time: ✅ 23.047ms (SLO: <24.100ms -4.4%) vs baseline: ~same

Memory: ✅ 50.428MB (SLO: <53.000MB -4.9%) vs baseline: +4.9%

✅ telemetryaddmetric - 30/30

✅ 1-count-metric-1-times

Time: ✅ 3.257µs (SLO: <20.000µs 📉 -83.7%) vs baseline: +1.5%

Memory: ✅ 31.339MB (SLO: <34.000MB -7.8%) vs baseline: +4.7%

✅ 1-count-metrics-100-times

Time: ✅ 223.495µs (SLO: <250.000µs 📉 -10.6%) vs baseline: +3.2%

Memory: ✅ 31.300MB (SLO: <34.000MB -7.9%) vs baseline: +5.2%

✅ 1-distribution-metric-1-times

Time: ✅ 3.034µs (SLO: <20.000µs 📉 -84.8%) vs baseline: +1.2%

Memory: ✅ 31.320MB (SLO: <34.000MB -7.9%) vs baseline: +4.8%

✅ 1-distribution-metrics-100-times

Time: ✅ 199.981µs (SLO: <220.000µs -9.1%) vs baseline: +0.4%

Memory: ✅ 31.320MB (SLO: <34.000MB -7.9%) vs baseline: +4.7%

✅ 1-gauge-metric-1-times

Time: ✅ 2.092µs (SLO: <20.000µs 📉 -89.5%) vs baseline: -1.5%

Memory: ✅ 31.221MB (SLO: <34.000MB -8.2%) vs baseline: +4.4%

✅ 1-gauge-metrics-100-times

Time: ✅ 124.368µs (SLO: <150.000µs 📉 -17.1%) vs baseline: -0.8%

Memory: ✅ 31.379MB (SLO: <34.000MB -7.7%) vs baseline: +5.1%

✅ 1-rate-metric-1-times

Time: ✅ 3.279µs (SLO: <20.000µs 📉 -83.6%) vs baseline: +1.9%

Memory: ✅ 31.320MB (SLO: <34.000MB -7.9%) vs baseline: +4.8%

✅ 1-rate-metrics-100-times

Time: ✅ 223.493µs (SLO: <250.000µs 📉 -10.6%) vs baseline: +3.5%

Memory: ✅ 31.280MB (SLO: <34.000MB -8.0%) vs baseline: +4.7%

✅ 100-count-metrics-100-times

Time: ✅ 21.960ms (SLO: <23.500ms -6.6%) vs baseline: +0.4%

Memory: ✅ 31.398MB (SLO: <34.000MB -7.7%) vs baseline: +4.9%

✅ 100-distribution-metrics-100-times

Time: ✅ 2.086ms (SLO: <2.250ms -7.3%) vs baseline: +3.7%

Memory: ✅ 31.379MB (SLO: <34.000MB -7.7%) vs baseline: +5.1%

✅ 100-gauge-metrics-100-times

Time: ✅ 1.284ms (SLO: <1.550ms 📉 -17.2%) vs baseline: -0.7%

Memory: ✅ 31.261MB (SLO: <34.000MB -8.1%) vs baseline: +4.7%

✅ 100-rate-metrics-100-times

Time: ✅ 2.265ms (SLO: <2.550ms 📉 -11.2%) vs baseline: +2.1%

Memory: ✅ 31.241MB (SLO: <34.000MB -8.1%) vs baseline: +4.7%

✅ flush-1-metric

Time: ✅ 4.630µs (SLO: <20.000µs 📉 -76.9%) vs baseline: +8.0%

Memory: ✅ 31.752MB (SLO: <34.000MB -6.6%) vs baseline: +5.0%

✅ flush-100-metrics

Time: ✅ 187.905µs (SLO: <250.000µs 📉 -24.8%) vs baseline: +0.9%

Memory: ✅ 31.693MB (SLO: <34.000MB -6.8%) vs baseline: +4.8%

✅ flush-1000-metrics

Time: ✅ 2.233ms (SLO: <2.500ms 📉 -10.7%) vs baseline: -0.2%

Memory: ✅ 32.539MB (SLO: <34.500MB -5.7%) vs baseline: +4.9%

✅ tracer - 6/6

✅ large

Time: ✅ 30.645ms (SLO: <32.950ms -7.0%) vs baseline: +0.4%

Memory: ✅ 32.971MB (SLO: <34.500MB -4.4%) vs baseline: +5.0%

✅ medium

Time: ✅ 2.992ms (SLO: <3.200ms -6.5%) vs baseline: +0.2%

Memory: ✅ 31.359MB (SLO: <34.000MB -7.8%) vs baseline: +4.9%

✅ small

Time: ✅ 339.390µs (SLO: <370.000µs -8.3%) vs baseline: +0.8%

Memory: ✅ 31.300MB (SLO: <34.000MB -7.9%) vs baseline: +4.6%

ℹ️ Scenarios Missing SLO Configuration (14 scenarios)

The following scenarios exist in candidate data but have no SLO thresholds configured:

  • coreapiscenario-context_with_data_listeners_and_all_listeners
  • coreapiscenario-core_dispatch_listeners
  • coreapiscenario-core_dispatch_listeners_and_all_listeners
  • coreapiscenario-core_dispatch_no_listeners
  • coreapiscenario-core_dispatch_only_all_listeners
  • coreapiscenario-core_dispatch_with_results_listeners
  • coreapiscenario-core_dispatch_with_results_listeners_and_all_listeners
  • coreapiscenario-core_dispatch_with_results_no_listeners
  • coreapiscenario-core_dispatch_with_results_only_all_listeners
  • djangosimple-baseline
  • errortrackingdjangosimple-baseline
  • errortrackingflasksqli-baseline
  • flasksimple-baseline
  • flasksqli-baseline

@brettlangdon brettlangdon merged commit d2812f2 into main Sep 18, 2025
400 of 402 checks passed
@brettlangdon brettlangdon deleted the juanjux/sourcecode-use-git-env-vars branch September 18, 2025 08:51
dd-octo-sts bot pushed a commit that referenced this pull request Sep 18, 2025
@juanjux @github-actions
fix: check git env vars before using the command on the source code i…
ebd9b7a 
…ntegration (#14613)

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

---------

Signed-off-by: Juanjo Alvarez <[email protected]>
(cherry picked from commit d2812f2)
@dd-octo-sts dd-octo-sts bot mentioned this pull request Sep 18, 2025
Merged
2 tasks
dd-octo-sts bot pushed a commit that referenced this pull request Sep 18, 2025
@juanjux @github-actions
fix: check git env vars before using the command on the source code i…
2dac030 
…ntegration (#14613)

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

---------

Signed-off-by: Juanjo Alvarez <[email protected]>
(cherry picked from commit d2812f2)
@dd-octo-sts dd-octo-sts bot mentioned this pull request Sep 18, 2025
Merged
2 tasks
dd-octo-sts bot pushed a commit that referenced this pull request Sep 18, 2025
@juanjux @github-actions
fix: check git env vars before using the command on the source code i…
fcf246f 
…ntegration (#14613)

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

---------

Signed-off-by: Juanjo Alvarez <[email protected]>
(cherry picked from commit d2812f2)
@dd-octo-sts dd-octo-sts bot mentioned this pull request Sep 18, 2025
Merged
2 tasks
juanjux added a commit that referenced this pull request Sep 18, 2025
@juanjux
fix: check git env vars before using the command on the source code i…
60406d9 
…ntegration (#14613)

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

---------

Signed-off-by: Juanjo Alvarez <[email protected]>
(cherry picked from commit d2812f2)
Signed-off-by: Juanjo Alvarez <[email protected]>
juanjux added a commit that referenced this pull request Sep 18, 2025
@juanjux
fix: check git env vars before using the command on the source code i…
c904f28 
…ntegration (#14613)

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

---------

Signed-off-by: Juanjo Alvarez <[email protected]>
(cherry picked from commit d2812f2)
Signed-off-by: Juanjo Alvarez <[email protected]>
juanjux added a commit that referenced this pull request Sep 18, 2025
@juanjux
fix: check git env vars before using the command on the source code i…
56518bc 
…ntegration (#14613)

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

---------

Signed-off-by: Juanjo Alvarez <[email protected]>
(cherry picked from commit d2812f2)
Signed-off-by: Juanjo Alvarez <[email protected]>
juanjux added a commit that referenced this pull request Sep 18, 2025
@dd-octo-sts @juanjux
fix: check git env vars before using the command on the source code i…
78d13a5 
…ntegration [backport 3.12] (#14619)

Backport d2812f2 from #14613 to 3.12.

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

Signed-off-by: Juanjo Alvarez <[email protected]>
Co-authored-by: Juanjo Alvarez Martinez <[email protected]>
juanjux added a commit that referenced this pull request Sep 18, 2025
@dd-octo-sts @juanjux
fix: check git env vars before using the command on the source code i…
78fc4f3 
…ntegration [backport 3.13] (#14620)

Backport d2812f2 from #14613 to 3.13.

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

Signed-off-by: Juanjo Alvarez <[email protected]>
Co-authored-by: Juanjo Alvarez Martinez <[email protected]>
juanjux added a commit that referenced this pull request Sep 18, 2025
@dd-octo-sts @juanjux
fix: check git env vars before using the command on the source code i…
69c197c 
…ntegration [backport 3.14] (#14621)

Backport d2812f2 from #14613 to 3.14.

The sourcecode integration was using git to retrieve the commit hash and
repository url. However, we already have defined `DD_GIT_COMMIT_HASH`
and `DD_GIT_REPOSITORY_URL` for these cases, so check these vars before.

## 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

Signed-off-by: Juanjo Alvarez <[email protected]>
Co-authored-by: Juanjo Alvarez Martinez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emmettbutler emmettbutler emmettbutler approved these changes

@brettlangdon brettlangdon brettlangdon approved these changes

@P403n1x87 P403n1x87 Awaiting requested review from P403n1x87 P403n1x87 is a code owner automatically assigned from DataDog/apm-python

@gnufede gnufede Awaiting requested review from gnufede

Assignees

@juanjux juanjux

Labels
backport 3.12 backport 3.13 backport 3.14
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@juanjux @emmettbutler @brettlangdon