MirrorHub/synapse
0
0
Fork 1
mirror of https://mau.dev/maunium/synapse.git synced 2024-06-18 02:28:21 +02:00
Code Issues Projects Releases Wiki Activity

Fix exception in background metrics collection

Browse source 
We attempted to iterate through a list on a separate thread without
doing the necessary copying.
This commit is contained in:
Erik Johnston 2018-10-03 11:28:01 +01:00
parent 25baf3b2ac
commit 7c570bff74
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
synapse/metrics/background_process_metrics.py
View file

@ -101,9 +101,13 @@ class _Collector(object):
labels=["name"],
)
# We copy the dict so that it doesn't change from underneath us
# We copy the dict so that it doesn't change from underneath us.
# We also copy the process lists as that can also change
with _bg_metrics_lock:
_background_processes_copy = dict(_background_processes)
_background_processes_copy = {
k: list(v)
for k, v in six.iteritems(_background_processes)
}
for desc, processes in six.iteritems(_background_processes_copy):
background_process_in_flight_count.add_metric(