Skip to content

Conversation

@gnufede
Copy link
Member

@gnufede gnufede commented Mar 21, 2025

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

@gnufede gnufede changed the title chore: use envier chore(ci_visibility): use envier Mar 21, 2025
@gnufede gnufede added changelog/no-changelog A changelog entry is not required for this PR. CI App labels Mar 21, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Mar 21, 2025

CODEOWNERS have been resolved as:

ddtrace/internal/ci_visibility/settings.py                              @DataDog/ci-app-libraries
ddtrace/contrib/internal/selenium/patch.py                              @DataDog/apm-core-python @DataDog/apm-idm-python
ddtrace/contrib/internal/unittest/patch.py                              @DataDog/ci-app-libraries
ddtrace/ext/test_visibility/__init__.py                                 @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/recorder.py                              @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/writer.py                                @DataDog/ci-app-libraries
tests/ci_visibility/api_client/test_ci_visibility_api_client.py         @DataDog/ci-app-libraries
tests/ci_visibility/test_ci_visibility.py                               @DataDog/ci-app-libraries
tests/ci_visibility/util.py                                             @DataDog/ci-app-libraries
tests/contrib/pytest/test_pytest.py                                     @DataDog/ci-app-libraries
tests/contrib/pytest/test_pytest_atr.py                                 @DataDog/ci-app-libraries

@github-actions
Copy link
Contributor

github-actions bot commented Mar 21, 2025

Bootstrap import analysis

Comparison of import times between this PR and main.

Summary

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

The average import time in main is: 246 ± 5 ms.

The import time difference between this PR and main is: -7.4 ± 0.2 ms.

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 3.408 ms (1.43%)
ddtrace.bootstrap.sitecustomize 2.024 ms (0.85%)
ddtrace.bootstrap.preload 1.937 ms (0.81%)
ddtrace.internal.products 1.724 ms (0.72%)
ddtrace.internal.remoteconfig.client 0.762 ms (0.32%)
ddtrace.internal.symbol_db.remoteconfig 0.185 ms (0.08%)
multiprocessing 0.164 ms (0.07%)
multiprocessing.context 0.164 ms (0.07%)
multiprocessing.reduction 0.164 ms (0.07%)
pickle 0.164 ms (0.07%)
_compat_pickle 0.164 ms (0.07%)
multiprocessing.sharedctypes 0.021 ms (0.01%)
multiprocessing.heap 0.021 ms (0.01%)
ddtrace.settings.profiling 0.212 ms (0.09%)
ddtrace.vendor.psutil 0.212 ms (0.09%)
ddtrace.vendor.psutil._pslinux 0.212 ms (0.09%)
ddtrace._trace.trace_handlers 0.048 ms (0.02%)
ddtrace._trace.utils_botocore.span_tags 0.048 ms (0.02%)
ddtrace._trace.utils_botocore.aws_payload_tagging 0.048 ms (0.02%)
ddtrace.vendor.jsonpath_ng 0.048 ms (0.02%)
ddtrace.vendor.jsonpath_ng.parser 0.048 ms (0.02%)
ddtrace.vendor.ply.yacc 0.048 ms (0.02%)
shlex 0.021 ms (0.01%)
ddtrace.appsec._common_module_patches 0.018 ms (0.01%)
ddtrace 1.383 ms (0.58%)
ddtrace._logger 0.433 ms (0.18%)
ddtrace.settings._core 0.318 ms (0.13%)
ddtrace.settings 0.318 ms (0.13%)
ddtrace.settings._config 0.318 ms (0.13%)
ddtrace.settings._telemetry 0.190 ms (0.08%)
ddtrace.internal.telemetry 0.190 ms (0.08%)
ddtrace.internal.telemetry.writer 0.190 ms (0.08%)
ddtrace.internal.utils.version 0.104 ms (0.04%)
ddtrace.vendor.packaging.version 0.104 ms (0.04%)
ddtrace.internal.telemetry.data 0.086 ms (0.04%)
ddtrace.internal.packages 0.053 ms (0.02%)
_sysconfigdata__linux_x86_64-linux-gnu 0.053 ms (0.02%)
sysconfig 0.033 ms (0.01%)
ddtrace.internal.gitmetadata 0.128 ms (0.05%)
ddtrace.ext.ci 0.128 ms (0.05%)
platform 0.128 ms (0.05%)
logging 0.116 ms (0.05%)
traceback 0.116 ms (0.05%)
contextlib 0.116 ms (0.05%)
ddtrace.trace 0.079 ms (0.03%)
ddtrace._trace.filters 0.031 ms (0.01%)
ddtrace._trace.processor 0.031 ms (0.01%)
ddtrace._trace.sampler 0.031 ms (0.01%)
ddtrace._trace.span 0.031 ms (0.01%)
ddtrace.internal._rand 0.031 ms (0.01%)
ddtrace._monkey 0.024 ms (0.01%)

@pr-commenter
Copy link

pr-commenter bot commented Mar 21, 2025

Benchmarks

Benchmark execution time: 2025-03-21 17:33:56

Comparing candidate commit 8cb05a5 in PR branch gnufede/ci-vis-envier with baseline commit 00c254b in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 236 metrics, 2 unstable metrics.

@github-actions github-actions bot added the stale label Apr 24, 2025
@github-actions
Copy link
Contributor

This pull request has been automatically closed after a period of inactivity.
After this much time, it will likely be easier to open a new pull request with the
same changes than to update this one from the base branch. Please comment or reopen
if you think this pull request was closed in error.

@github-actions github-actions bot closed this Apr 26, 2025
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. CI App stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants