diff options
| author | Amber Brown <hawkowl@atleastfornow.net> | 2018-07-19 20:58:18 +1000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-07-19 20:58:18 +1000 |
| commit | 95ccb6e2ec57f2150a697ea9cde030e8f78d6db9 (patch) | |
| tree | 7ef62670b02e1d55d3c79bdffdaecba9ddd76ebc /synapse/util/metrics.py | |
| parent | Merge pull request #3507 from Peetz0r/patch-1 (diff) | |
| download | synapse-95ccb6e2ec57f2150a697ea9cde030e8f78d6db9.tar.xz | |
Don't spew errors because we can't save metrics (#3563)
Diffstat (limited to 'synapse/util/metrics.py')
| -rw-r--r-- | synapse/util/metrics.py | 19 |
1 files changed, 13 insertions, 6 deletions
diff --git a/synapse/util/metrics.py b/synapse/util/metrics.py index 6ba7107896..97f1267380 100644 --- a/synapse/util/metrics.py +++ b/synapse/util/metrics.py @@ -104,12 +104,19 @@ class Measure(object): logger.warn("Expected context. (%r)", self.name) return - usage = context.get_resource_usage() - self.start_usage - block_ru_utime.labels(self.name).inc(usage.ru_utime) - block_ru_stime.labels(self.name).inc(usage.ru_stime) - block_db_txn_count.labels(self.name).inc(usage.db_txn_count) - block_db_txn_duration.labels(self.name).inc(usage.db_txn_duration_sec) - block_db_sched_duration.labels(self.name).inc(usage.db_sched_duration_sec) + current = context.get_resource_usage() + usage = current - self.start_usage + try: + block_ru_utime.labels(self.name).inc(usage.ru_utime) + block_ru_stime.labels(self.name).inc(usage.ru_stime) + block_db_txn_count.labels(self.name).inc(usage.db_txn_count) + block_db_txn_duration.labels(self.name).inc(usage.db_txn_duration_sec) + block_db_sched_duration.labels(self.name).inc(usage.db_sched_duration_sec) + except ValueError: + logger.warn( + "Failed to save metrics! OLD: %r, NEW: %r", + self.start_usage, current + ) if self.created_context: self.start_context.__exit__(exc_type, exc_val, exc_tb) |