Skip to content

ci: use dd-octo-sts in package job #13948

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 3 commits into from
Jul 14, 2025
Merged

ci: use dd-octo-sts in package job #13948

merged 3 commits into from
Jul 14, 2025
+6 −2

Conversation

brettlangdon
Copy link
Member

@brettlangdon brettlangdon commented Jul 10, 2025 •
edited
Loading

LANGPLAT-661

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

@github-actions GitHub Actions
Copy link
Contributor

CODEOWNERS have been resolved as:

.gitlab/package.yml                                                     @DataDog/python-guild @DataDog/apm-core-python

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jul 10, 2025 •
edited
Loading

Bootstrap import analysis

Comparison of import times between this PR and base.

Summary

The average import time from this PR is: 278 ± 2 ms.

The average import time from base is: 280 ± 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 1.992 ms (0.72%)
ddtrace.bootstrap.sitecustomize 1.319 ms (0.47%)
ddtrace.bootstrap.preload 1.319 ms (0.47%)
ddtrace.internal.remoteconfig.client 0.644 ms (0.23%)
ddtrace 0.673 ms (0.24%)
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 changed the title use dd-octo-sts for package jobs ci: use dd-octo-sts in GitLab jobs Jul 11, 2025
@brettlangdon brettlangdon added the changelog/no-changelog A changelog entry is not required for this PR. label Jul 11, 2025
@brettlangdon brettlangdon marked this pull request as ready for review July 11, 2025 16:36
@brettlangdon brettlangdon requested review from a team as code owners July 11, 2025 16:36
@brettlangdon brettlangdon requested review from wantsui and juanjux July 11, 2025 16:36
@pr-commenter
Copy link

pr-commenter bot commented Jul 11, 2025 •
edited
Loading

Benchmarks

Benchmark execution time: 2025-07-11 17:33:28

Comparing candidate commit 233c4ff in PR branch brettlangdon/use.octo-sts with baseline commit 3cfd051 in branch main.

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

@brettlangdon brettlangdon mentioned this pull request Jul 11, 2025
Open
2 tasks
@brettlangdon brettlangdon changed the title ci: use dd-octo-sts in GitLab jobs ci: use dd-octo-sts in package job Jul 14, 2025
@brettlangdon brettlangdon enabled auto-merge (squash) July 14, 2025 12:29
@brettlangdon brettlangdon merged commit e0a772c into main Jul 14, 2025
881 of 882 checks passed
@brettlangdon brettlangdon deleted the brettlangdon/use.octo-sts branch July 14, 2025 12:34
taegyunkim pushed a commit that referenced this pull request Jul 14, 2025
@brettlangdon @taegyunkim
ci: use dd-octo-sts in package job (#13948)
e8a1390 
LANGPLAT-661

## 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)
github-actions bot pushed a commit that referenced this pull request Jul 15, 2025
@brettlangdon @github-actions
ci: use dd-octo-sts in package job (#13948)
18caca9 
LANGPLAT-661

## 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)

(cherry picked from commit e0a772c)
@github-actions github-actions bot mentioned this pull request Jul 15, 2025
Open
2 tasks
brettlangdon added a commit that referenced this pull request Jul 15, 2025
@brettlangdon
ci: use dd-octo-sts in package job [backport 3.10] (#14006)
9bc91d6 
Backport e0a772c from #13948 to 3.10.

LANGPLAT-661

- [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))

- [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)

Co-authored-by: Brett Langdon <brett.langdon@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juanjux juanjux juanjux approved these changes

@wantsui wantsui Awaiting requested review from wantsui wantsui is a code owner automatically assigned from DataDog/python-guild

Assignees
No one assigned
Labels
backport 3.10 changelog/no-changelog A changelog entry is not required for this PR.
Projects
None yet
Milestone
No milestone
Development

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