Skip to content
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.

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

Update dependencies #16788

Merged
merged 2 commits into from
Feb 6, 2024
Merged

Update dependencies #16788

merged 2 commits into from
Feb 6, 2024

Conversation

FlorentClarret
Copy link
Member

@FlorentClarret FlorentClarret commented Feb 5, 2024

What does this PR do?

Update most of our dependencies except psutil

Motivation

Agent build is failing for psutil, I'll investigate in a separate PR

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Copy link

github-actions bot commented Feb 5, 2024

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

Copy link

codecov bot commented Feb 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (59abdb2) 88.46% compared to head (ee71517) 88.58%.
Report is 7 commits behind head on master.

Additional details and impacted files
Flag Coverage Δ
datadog_checks_base 80.74% <ø> (+0.82%) ⬆️
jboss_wildfly ?
process 85.28% <ø> (+0.28%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@FlorentClarret FlorentClarret force-pushed the florentclarret/bump_deps branch from a11616f to ffadc09 Compare February 5, 2024 10:32
Copy link

github-actions bot commented Feb 5, 2024

Test Results

   119 files    119 suites   1h 10m 37s ⏱️
 2 901 tests 2 878 ✅    23 💤 0 ❌
12 223 runs  9 427 ✅ 2 796 💤 0 ❌

Results for commit ee71517.

♻️ This comment has been updated with latest results.

@FlorentClarret FlorentClarret marked this pull request as ready for review February 5, 2024 13:07
@FlorentClarret FlorentClarret requested review from a team as code owners February 5, 2024 13:07
@@ -40,7 +40,8 @@ license = "BSD-3-Clause"
[project.optional-dependencies]
deps = [
"futures==3.4.0; python_version < '3.0'",
"pyvmomi==8.0.2.0.1",
"pyvmomi==8.0.0.1; python_version < '3.0'",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!!

@FlorentClarret FlorentClarret merged commit 8f40605 into master Feb 6, 2024
90 checks passed
@FlorentClarret FlorentClarret deleted the florentclarret/bump_deps branch February 6, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment