]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
FIP debug messages
authorMichael Smith <michael.smith6@hp.com>
Tue, 24 Feb 2015 00:21:54 +0000 (16:21 -0800)
committerMichael Smith <michael.smith6@hp.com>
Mon, 9 Mar 2015 15:40:14 +0000 (15:40 +0000)
The experimental DVR test job is seeing intermittent failures
related to FIPs.  This patch will add debug log messages to the
l3-agent to help debug the failures.  These logs messages will
indicate when fips have been added and the fip statuses before
they are sent to the plugin.

Change-Id: Ic27c02ae45971efc9ba2aec84aafaa89ff9243e6

neutron/agent/l3/agent.py
neutron/agent/l3/router_info.py

index 8ac4b02b21672ec6201e7df1d12f0aa681bad0f7..a8136e27abd5e34f236c1511fd33589dea94a169 100644 (file)
@@ -435,6 +435,7 @@ class L3NATAgent(firewall_l3_agent.FWaaSL3AgentRpcCallback,
         ri.floating_ips = set(fip_statuses.keys())
         for fip_id in existing_floating_ips - ri.floating_ips:
             fip_statuses[fip_id] = l3_constants.FLOATINGIP_STATUS_DOWN
+        LOG.debug('Sending floating ip statuses: %s', fip_statuses)
         # Update floating IP status on the neutron server
         self.plugin_rpc.update_floatingip_statuses(
             self.context, ri.router_id, fip_statuses)
index 9e0034ff9863a5a4f0c8c6b3a1acecad448f93b8..d068d3c5fb2f4a3afa254eb98feefe0d94b7a291 100644 (file)
@@ -214,6 +214,9 @@ class RouterInfo(object):
             if ip_cidr not in existing_cidrs:
                 fip_statuses[fip['id']] = self.add_floating_ip(
                     fip, interface_name, device)
+                LOG.debug('Floating ip %(id)s added, status %(status)s',
+                          {'id': fip['id'],
+                           'status': fip_statuses.get(fip['id'])})
 
         fips_to_remove = (
             ip_cidr for ip_cidr in existing_cidrs - new_cidrs