]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Change log level from error to warning
authorzoukeke@cmss.chinamobile.com <zoukeke@cmss.chinamobile.com>
Wed, 23 Dec 2015 13:15:51 +0000 (21:15 +0800)
committerzoukeke@cmss.chinamobile.com <zoukeke@cmss.chinamobile.com>
Thu, 24 Dec 2015 04:51:26 +0000 (12:51 +0800)
The service respawning action of ProcessMonitor is normal handling. So
the log level in external_process.py should not be error.

Closes-Bug: #1528850
Change-Id: Ie335f521b09310a577eb4fb03946dfba484eef61

neutron/agent/linux/external_process.py

index 42a302f07973461a893d5536d7ef1279355fbefd..fc969e51714ec2b1bce13ab961def2b351f8b9f9 100644 (file)
@@ -23,7 +23,7 @@ from oslo_config import cfg
 from oslo_log import log as logging
 from oslo_utils import fileutils
 
-from neutron._i18n import _, _LE
+from neutron._i18n import _, _LW, _LE
 from neutron.agent.common import config as agent_cfg
 from neutron.agent.linux import ip_lib
 from neutron.agent.linux import utils
@@ -247,9 +247,9 @@ class ProcessMonitor(object):
         action_function(service_id)
 
     def _respawn_action(self, service_id):
-        LOG.error(_LE("Respawning %(service)s for uuid %(uuid)s"),
-                  {'service': service_id.service,
-                   'uuid': service_id.uuid})
+        LOG.warning(_LW("Respawning %(service)s for uuid %(uuid)s"),
+                    {'service': service_id.service,
+                     'uuid': service_id.uuid})
         self._monitored_processes[service_id].enable()
 
     def _exit_action(self, service_id):