Skip to content

Commit

Permalink
Unpin psutil dependency (#790)
Browse files Browse the repository at this point in the history
* Unpin psutil dependency
* Update test for new psutil behavior

It is unlikely, but if the internal process cache is not
cleared, we could have a stale result.

---------

Co-authored-by: Lance T. Erickson <[email protected]>
  • Loading branch information
tim-schilling and lancetarn authored Oct 4, 2024
1 parent 5614351 commit db6a1f0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@
},
install_requires=[
"asgiref",
"psutil>=5,<6",
"psutil>=5",
"urllib3",
"certifi",
"wrapt>=1.10,<2.0",
Expand Down
6 changes: 5 additions & 1 deletion tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,11 @@ def core_agent_manager(core_agent_dir):


def core_agent_is_running():
return any(p.name() == "core-agent" for p in psutil.process_iter(["name"]))
return any(
p.name() == "core-agent"
for p in psutil.process_iter(["name"])
if p.is_running()
)


def terminate_core_agent_processes():
Expand Down

0 comments on commit db6a1f0

Please sign in to comment.