From 5f8a1fbf804c9572b777eb6af3d24cf9a98df758 Mon Sep 17 00:00:00 2001 From: Eoghan Glynn Date: Tue, 10 Jun 2014 17:07:32 +0100 Subject: [PATCH] Remove low-value logging from publication codepath Previously, the pipeline publish codepath emitted log messages with almost no diagnostic value other than a "comfort bump". However the sheer volume of these message was overwhelming logstash in the CI gate. We remove the worst offenders from that codepath. Change-Id: If630593e8f755ccc1ede16941459daac30d284ff Closes-Bug: #1328596 --- ceilometer/pipeline.py | 2 -- ceilometer/publisher/rpc.py | 2 -- 2 files changed, 4 deletions(-) diff --git a/ceilometer/pipeline.py b/ceilometer/pipeline.py index fc9eeb37b..6126798bc 100644 --- a/ceilometer/pipeline.py +++ b/ceilometer/pipeline.py @@ -303,7 +303,6 @@ class Sink(object): transformed_samples.append(sample) if transformed_samples: - LOG.audit(_("Pipeline %s: Publishing samples"), self) for p in self.publishers: try: p.publish_samples(ctxt, transformed_samples) @@ -312,7 +311,6 @@ class Sink(object): "Pipeline %(pipeline)s: Continue after error " "from publisher %(pub)s") % ({'pipeline': self, 'pub': p})) - LOG.audit(_("Pipeline %s: Published samples") % self) def publish_samples(self, ctxt, samples): for meter_name, samples in itertools.groupby( diff --git a/ceilometer/publisher/rpc.py b/ceilometer/publisher/rpc.py index 517580975..f62cb4f04 100644 --- a/ceilometer/publisher/rpc.py +++ b/ceilometer/publisher/rpc.py @@ -124,8 +124,6 @@ class RPCPublisher(publisher.PublisherBase): ] topic = cfg.CONF.publisher_rpc.metering_topic - LOG.audit(_('Publishing %(m)d samples on %(t)s') % ( - {'m': len(meters), 't': topic})) self.local_queue.append((context, topic, meters)) if self.per_meter_topic: