Update dependencies - #24958
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
Warning This PR changes Agent dependencies. Do not merge it until the wheels are promoted. Dependency inputs changed, so the dependency resolution and wheel build workflow runs on this branch. It takes 1.5 to 3 hours and finishes by committing updated lockfiles. The wheels it builds then have to be promoted before the Agent can use them. Steps to take, in order:
Merging before promotion leaves the Agent unable to install the wheels these lockfiles pin. Pushing again to this branch invalidates the promotion, so if you push after step 4 you have to promote again. Head commit |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 25f0d66f39
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| @@ -0,0 +1 @@ | |||
| Update dependencies No newline at end of file | |||
There was a problem hiding this comment.
End dependency changelog entries with periods
This fragment, along with the eight other changelog entries added in this commit, omits the required terminal period. Add a period to every generated Update dependencies entry so the released changelogs follow the repository’s required format.
AGENTS.md reference: AGENTS.md:L208-L211
Useful? React with 👍 / 👎.
evalya-impact-summaryevalya impact analysis |
|
There was a problem hiding this comment.
The changelog files use PR number 24794, and the dependency resolution files still use the old package versions. These errors stop the required checks.
🤖 Datadog Autotest · Commit 4da33c8 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
| @@ -0,0 +1 @@ | |||
| Update dependencies | |||
There was a problem hiding this comment.
Use PR number 24958 in changelog names
The changelog validation check fails and blocks the PR.
Assertion details
- Input: Run the PR validation for PR 24958 with the added
24794.addedfiles. - Expected:
Each new changelog file must use PR number 24958. - Actual:
All nine new files use PR number 24794. The changelog validator compares each file name with PR number 24958 and reports an error.
Was this helpful? React 👍 or 👎
🤖 Datadog Autotest · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
| binary==1.0.2 | ||
| boto3==1.43.67 | ||
| botocore==1.43.67 | ||
| boto3==1.43.78 |
There was a problem hiding this comment.
Update the generated dependency resolution
The dependency resolution check fails and blocks the merge queue.
Assertion details
- Input: Run the required resolution hash check after the dependency version changes.
- Expected:
The dependency workflow must update the resolution hash and all generated resolution files for the new versions. - Actual: The PR changes a dependency hash input, but
.deps/builder_inputs.tomlkeeps the old hash. The resolved files also keep the old package versions.
Was this helpful? React 👍 or 👎
🤖 Datadog Autotest · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
There was a problem hiding this comment.
More details
The new dependency versions agree in the project files and generated dependency files. The static review finds no concrete defect.
🤖 Datadog Autotest · Commit b4dff0d · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
Validation ReportAll 21 validations passed. Show details
|
What does this PR do?
Update the dependencies
Dependency Bumps
This PR was automatically generated by the following workflow:
https://github.com/DataDog/integrations-core/actions/runs/32683017668
Review checklist (to be filled by reviewers)
qa/skip-qalabel if the PR doesn't need to be tested during QA.