openstack/swift
Code Issues Proposed changes

Merge "proxy_logging: remove unnecessary condition"

Browse Source
This commit is contained in:
Zuul
2025-05-13 19:12:00 +00:00
committed by Gerrit Code Review
parent 2c5bcfdf0a 11257eda5a
commit 6212869399
1 changed files with 4 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
swift/common/middleware/proxy_logging.py
View File

@@ -333,14 +333,10 @@ class ProxyLoggingMiddleware(object):
metric_name_policy = self.statsd_metric_name_policy(req, status_int, metric_name_policy = self.statsd_metric_name_policy(req, status_int,
method, method,
policy_index) policy_index)
# Only log data for valid controllers (or SOS) to keep the metric count self.access_logger.timing(metric_name + '.timing',
# down (egregious errors will get logged by the proxy server itself). (end_time - start_time) * 1000)
self.access_logger.update_stats(metric_name + '.xfer',
if metric_name: bytes_received + bytes_sent)
self.access_logger.timing(metric_name + '.timing',
(end_time - start_time) * 1000)
self.access_logger.update_stats(metric_name + '.xfer',
bytes_received + bytes_sent)
if metric_name_policy: if metric_name_policy:
self.access_logger.timing(metric_name_policy + '.timing', self.access_logger.timing(metric_name_policy + '.timing',
(end_time - start_time) * 1000) (end_time - start_time) * 1000)