From c000ae3c1e1c9f0e096fd1bc5bd44305a752ade2 Mon Sep 17 00:00:00 2001 From: LIU Yulong Date: Tue, 10 Nov 2015 16:27:04 +0800 Subject: [PATCH] Correct insufficient name for external process in manager log Let's just make the log more precise. But remember the inconsitent use of the ProcessManager: The following external processes's ProcessManager did not set the 'service' param: 1. HA router IP monitor 2. DHCP dnsmasq 3. keepalived(vrrp) 4. metadata-proxy The following set: 1. dibbler 2. radvd Change-Id: I93b742ff1e52f15e5541ef3e7c8e844c70e8dd6c Closes-Bug: #1514728 --- neutron/agent/linux/external_process.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/neutron/agent/linux/external_process.py b/neutron/agent/linux/external_process.py index a1c6fd1a214..0b9ebf3f5c7 100644 --- a/neutron/agent/linux/external_process.py +++ b/neutron/agent/linux/external_process.py @@ -230,7 +230,7 @@ class ProcessMonitor(object): LOG.error(_LE("%(service)s for %(resource_type)s " "with uuid %(uuid)s not found. " "The process should not have died"), - {'service': pm.service, + {'service': service_id.service, 'resource_type': self._resource_type, 'uuid': service_id.uuid}) self._execute_action(service_id)