]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Cisco: unsupported format character in log format
authorAngus Lees <gus@inodes.org>
Wed, 17 Dec 2014 04:44:12 +0000 (15:44 +1100)
committerAngus Lees <gus@inodes.org>
Wed, 17 Dec 2014 04:44:12 +0000 (15:44 +1100)
CiscoCfgAgent.hosting_devices_removed() contains an error in the
format string used to log errors:

    LOG.error(_LE("Invalid payload format for received RPC message "
"`hosting_devices_removed`. Error is %{error}s. "
"Payload is %(payload)s"),
      {'error': e, 'payload': payload})

"%{error}s" should be "%(error)s"

The existing version raises "ValueError: unsupported format character
'{' (0x7b)" when invoked.

(found via pylint)

Change-Id: Ic5e9db2a19aa7652cbbd1ee04d7e02914e8d0093
Closes-Bug: #1403304

neutron/plugins/cisco/cfg_agent/cfg_agent.py

index dfc094aca7f577267a9ab8ce2482278c83315e7a..636fb5c8105a0b98ba48ddf8f9e7f52286fd9cb9 100644 (file)
@@ -228,7 +228,7 @@ class CiscoCfgAgent(manager.Manager):
                     self.process_services(removed_devices_info=payload)
         except KeyError as e:
             LOG.error(_LE("Invalid payload format for received RPC message "
-                        "`hosting_devices_removed`. Error is %{error}s. "
+                        "`hosting_devices_removed`. Error is %(error)s. "
                         "Payload is %(payload)s"),
                       {'error': e, 'payload': payload})