openstack/cinder
Code Issues Proposed changes

Log which service is down

Browse Source 
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
This commit is contained in:
Eric Harney
2015-07-27 13:24:36 -04:00
parent 70b6ee3ab3
commit 4f06fbe1cd
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
cinder/service.py
View File

@@ -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()