From: Eric Harney <eharney@redhat.com>
Date: Mon, 27 Jul 2015 17:24:36 +0000 (-0400)
Subject: Log which service is down
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=4f06fbe1cd351845c00d4f822d32eef1cf1989fe;p=openstack-build%2Fcinder-build.git

Log which service is down

This changes the message "Manager for service cinder-volume
is reporting problems..." to
"Manager for service cinder-volume myhost@backend1 is
reporting problems..."

This makes it more clear in the log file which backend this
message is referring to.

Change-Id: I62172080084a01c0324ffb6ab8a0e4af2b47fb85
---

diff --git a/cinder/service.py b/cinder/service.py
index 595e6da4c..95010fa05 100644
--- a/cinder/service.py
+++ b/cinder/service.py
@@ -293,9 +293,11 @@ class Service(service.Service):
         if not self.manager.is_working():
             # NOTE(dulek): If manager reports a problem we're not sending
             # heartbeats - to indicate that service is actually down.
-            LOG.error(_LE('Manager for service %s is reporting problems, skip '
-                          'sending heartbeat. Service will appear "down".'),
-                      self.binary)
+            LOG.error(_LE('Manager for service %(binary)s %(host)s is '
+                          'reporting problems, not sending heartbeat. '
+                          'Service will appear "down".'),
+                      {'binary': self.binary,
+                       'host': self.host})
             return
 
         ctxt = context.get_admin_context()