Skip to content

Conversation

@emmettbutler
Copy link
Collaborator

#15253 removed aioredis integration folders but did not touch the integration registry, this meant the checks on the integration registry never ran and didn't catch that we didn't remove aioredis integration entirely.

This change updates the definition for the "contrib" component to be more inclusive of all contrib files.


(cherry picked from commit be7f4cb)

#15253 removed aioredis
integration folders but did not touch the integration registry, this
meant the checks on the integration registry never ran and didn't catch
that we didn't remove aioredis integration entirely.

This change updates the definition for the "contrib" component to be
more inclusive of all contrib files.

<!-- Describe your testing strategy or note what tests are included -->

<!-- Note any risks associated with this change, or "None" if no risks
-->

<!-- Any other information that would be helpful for reviewers -->

---------

Co-authored-by: Emmett Butler <[email protected]>
(cherry picked from commit be7f4cb)
@github-actions
Copy link
Contributor

CODEOWNERS have been resolved as:

ddtrace/contrib/integration_registry/registry.yaml                      @DataDog/apm-core-python @DataDog/apm-idm-python
tests/contrib/suitespec.yml                                             @DataDog/apm-core-python @DataDog/apm-idm-python

@emmettbutler emmettbutler added the changelog/no-changelog A changelog entry is not required for this PR. label Nov 14, 2025
@emmettbutler emmettbutler enabled auto-merge (squash) November 14, 2025 18:01
@emmettbutler emmettbutler merged commit b2265b1 into 4.0 Nov 14, 2025
129 of 426 checks passed
@emmettbutler emmettbutler deleted the backport-15276-to-4.0 branch November 14, 2025 18:12
@emmettbutler
Copy link
Collaborator Author

force-merged to make the 4.0 branch consistent a little more quickly

@github-actions
Copy link
Contributor

github-actions bot commented Nov 14, 2025

Bootstrap import analysis

Comparison of import times between this PR and base.

Summary

The average import time from this PR is: 238 ± 3 ms.

The average import time from base is: 240 ± 3 ms.

The import time difference between this PR and base is: -1.9 ± 0.1 ms.

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 2.001 ms (0.84%)
ddtrace.bootstrap.sitecustomize 1.320 ms (0.55%)
ddtrace.bootstrap.preload 1.320 ms (0.55%)
ddtrace.internal.remoteconfig.client 0.625 ms (0.26%)
ddtrace 0.681 ms (0.29%)
ddtrace.internal._unpatched 0.027 ms (0.01%)
json 0.027 ms (0.01%)
json.decoder 0.027 ms (0.01%)
re 0.027 ms (0.01%)
enum 0.027 ms (0.01%)
types 0.027 ms (0.01%)

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.

3 participants