Skip to content

Bump psutil version to 5.9.7#16547

Merged
FlorentClarret merged 2 commits into
masterfrom
florentclarret/disk/psutil
Jan 15, 2024
Merged

Bump psutil version to 5.9.7#16547
FlorentClarret merged 2 commits into
masterfrom
florentclarret/disk/psutil

Conversation

@FlorentClarret

@FlorentClarret FlorentClarret commented Jan 8, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

What does this PR do?

Bump psutil version to 5.9.7

Motivation

One version broke our tests but it's not the case anymore. I'll drop it from the dep command in a follow-up 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.

@FlorentClarret FlorentClarret changed the title Bump psutil version to 5.9.7 Bump psutil version to 5.9.7 Jan 8, 2024
@github-actions

github-actions Bot commented Jan 8, 2024

Copy link
Copy Markdown
Contributor

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

@codecov

codecov Bot commented Jan 8, 2024 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a9d7338) 90.48% compared to head (07c77ed) 90.67%.
Report is 4 commits behind head on master.

❗ Current head 07c77ed differs from pull request most recent head d581698. Consider uploading reports for the commit d581698 to get more accurate results

Additional details and impacted files
Flag Coverage Δ
btrfs 82.91% <ø> (ø)
cassandra ?
disk 91.47% <ø> (+2.13%) ⬆️
gunicorn 92.10% <ø> (ø)
ibm_mq 91.28% <ø> (ø)
ignite ?
network 93.65% <ø> (+1.62%) ⬆️
process 85.28% <ø> (+0.28%) ⬆️
solr ?
system_core 92.66% <ø> (ø)
system_swap 98.30% <ø> (ø)

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

@FlorentClarret
FlorentClarret force-pushed the florentclarret/disk/psutil branch from e3e5ff8 to a0ea415 Compare January 8, 2024 07:45
@FlorentClarret FlorentClarret mentioned this pull request Jan 8, 2024
Merged
3 tasks
@github-actions

github-actions Bot commented Jan 8, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Test Results

 40 files   40 suites   3m 37s ⏱️
235 tests 235 ✅   0 💤 0 ❌
889 runs  768 ✅ 121 💤 0 ❌

Results for commit 07c77ed.

♻️ This comment has been updated with latest results.

@FlorentClarret
FlorentClarret marked this pull request as ready for review January 8, 2024 10:01
@FlorentClarret
FlorentClarret requested review from a team as code owners January 8, 2024 10:01
iliakur
iliakur previously approved these changes Jan 8, 2024
View reviewed changes
iliakur
iliakur reviewed Jan 8, 2024
View reviewed changes

@iliakur iliakur left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wait, shouldn't the this line also get updated?

@FlorentClarret

FlorentClarret commented Jan 8, 2024

Copy link
Copy Markdown
Member Author

Wait, shouldn't the this line also get updated?

You mean this? https://github.com/DataDog/integrations-core/pull/16547/files#diff-1b02d4258aa29155f48813d2eb55fc87f62abbded6d8a9fa9e3b8f08365d771aL60

julien-lebot
julien-lebot previously approved these changes Jan 8, 2024
View reviewed changes
@FlorentClarret FlorentClarret added this to the 7.52.0 milestone Jan 8, 2024
ofek
ofek previously approved these changes Jan 8, 2024
View reviewed changes
@FlorentClarret
FlorentClarret dismissed stale reviews from ofek, julien-lebot, and iliakur via d581698 January 15, 2024 07:46
@FlorentClarret
FlorentClarret force-pushed the florentclarret/disk/psutil branch from 07c77ed to d581698 Compare January 15, 2024 07:46
@FlorentClarret
FlorentClarret merged commit fde8722 into master Jan 15, 2024
@FlorentClarret
FlorentClarret deleted the florentclarret/disk/psutil branch January 15, 2024 07:53
FlorentClarret added a commit that referenced this pull request Mar 8, 2024
@FlorentClarret
Revert "Bump psutil version to 5.9.7 (#16547)"
70a39e0 
This reverts commit fde8722
@FlorentClarret FlorentClarret mentioned this pull request Mar 8, 2024
Merged
3 tasks
FlorentClarret added a commit that referenced this pull request Mar 8, 2024
@FlorentClarret
Revert "Bump psutil version to 5.9.7 (#16547)"
d3b50da 
This reverts commit fde8722
FlorentClarret added a commit that referenced this pull request Mar 8, 2024
@FlorentClarret
Revert "Bump psutil version to 5.9.7 (#16547)"
72e6d2a 
This reverts commit fde8722
FlorentClarret added a commit that referenced this pull request Mar 8, 2024
@FlorentClarret
Revert "Bump psutil version to 5.9.7 (#16547)"
2b16453 
This reverts commit fde8722
FlorentClarret added a commit that referenced this pull request Mar 8, 2024
@FlorentClarret
Revert "Bump psutil version to 5.9.7 (#16547)" (#17112)
ea412ad 
This reverts commit fde8722
@FlorentClarret FlorentClarret mentioned this pull request Mar 8, 2024
Merged
3 tasks
FlorentClarret added a commit that referenced this pull request Mar 8, 2024
@FlorentClarret
Revert "Bump psutil version to 5.9.7 (#16547)" (#17112)
63a957a 
This reverts commit fde8722
FlorentClarret added a commit that referenced this pull request Mar 8, 2024
@FlorentClarret
Revert "Bump psutil version to 5.9.7 (#16547)" (#17112) (#17118)
6f41dd3 
This reverts commit fde8722
marlon-merida pushed a commit that referenced this pull request Mar 11, 2024
@FlorentClarret @marlon-merida
Revert "Bump psutil version to 5.9.7 (#16547)" (#17112) (#17118)
d867f7b 
This reverts commit fde8722
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iliakur iliakur iliakur left review comments
@julien-lebot julien-lebot julien-lebot left review comments
@ofek ofek ofek left review comments

Assignees

No one assigned

Labels

base_package dependencies integration/btrfs integration/disk integration/gunicorn integration/ibm_mq integration/network integration/process integration/system_core integration/system_swap team/agent-integrations team/windows-agent

Projects

None yet

Milestone

7.52.0

Development

Successfully merging this pull request may close these issues.

4 participants

@FlorentClarret @julien-lebot @iliakur @ofek