Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Commit

Permalink
Make psutil an explicit dependency
Browse files Browse the repository at this point in the history
As of #4027, we require psutil to be installed, so it should be in our
dependency list. We can also remove some of the conditional import code
introduced by #992.

Fixes #4062.
  • Loading branch information
richvdh committed Oct 19, 2018
1 parent 81d4f51 commit e5b52d0
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 10 deletions.
1 change: 1 addition & 0 deletions changelog.d/4073.misc
@@ -0,0 +1 @@
Add psutil as an explicit dependency
11 changes: 4 additions & 7 deletions synapse/app/homeserver.py
Expand Up @@ -20,6 +20,7 @@

from six import iteritems

import psutil
from prometheus_client import Gauge

from twisted.application import service
Expand Down Expand Up @@ -502,20 +503,16 @@ def phone_stats_home():

def performance_stats_init():
try:
import psutil
process = psutil.Process()
# Ensure we can fetch both, and make the initial request for cpu_percent
# so the next request will use this as the initial point.
process.memory_info().rss
process.cpu_percent(interval=None)
logger.info("report_stats can use psutil")
stats_process.append(process)
except (ImportError, AttributeError):
logger.warn(
"report_stats enabled but psutil is not installed or incorrect version."
" Disabling reporting of memory/cpu stats."
" Ensuring psutil is available will help matrix.org track performance"
" changes across releases."
except (AttributeError):
logger.warning(
"Unable to read memory/cpu stats. Disabling reporting."
)

def generate_user_daily_visit_stats():
Expand Down
4 changes: 1 addition & 3 deletions synapse/python_dependencies.py
Expand Up @@ -53,6 +53,7 @@
"pillow>=3.1.2": ["PIL"],
"pydenticon>=0.2": ["pydenticon"],
"sortedcontainers>=1.4.4": ["sortedcontainers"],
"psutil>=2.0.0": ["psutil>=2.0.0"],
"pysaml2>=3.0.0": ["saml2"],
"pymacaroons-pynacl>=0.9.3": ["pymacaroons"],
"msgpack-python>=0.4.2": ["msgpack"],
Expand All @@ -79,9 +80,6 @@
"matrix-synapse-ldap3": {
"matrix-synapse-ldap3>=0.1": ["ldap_auth_provider"],
},
"psutil": {
"psutil>=2.0.0": ["psutil>=2.0.0"],
},
"postgres": {
"psycopg2>=2.6": ["psycopg2"]
}
Expand Down

0 comments on commit e5b52d0

Please sign in to comment.