From: Aaron Rosen <aaronorosen@gmail.com>
Date: Fri, 21 Mar 2014 04:13:48 +0000 (-0700)
Subject: Log received pool.status
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=af0c9ea3b8e8f5cc4fd523ff336887ae94bb6ef0;p=openstack-build%2Fneutron-build.git

Log received pool.status

This patch adds a log statement so we can figure out what status the
lbaas-agent is passing in. This way we can figure out what to prevent.

Change-Id: I1a0f809140127ccb9596d83c97add03a853991a3
Closes-bug: #1295491
---

diff --git a/neutron/services/loadbalancer/drivers/common/agent_driver_base.py b/neutron/services/loadbalancer/drivers/common/agent_driver_base.py
index ac352fadd..229024bae 100644
--- a/neutron/services/loadbalancer/drivers/common/agent_driver_base.py
+++ b/neutron/services/loadbalancer/drivers/common/agent_driver_base.py
@@ -94,9 +94,11 @@ class LoadBalancerCallbacks(object):
             qry = context.session.query(loadbalancer_db.Pool)
             qry = qry.filter_by(id=pool_id)
             pool = qry.one()
-
             if pool.status != constants.ACTIVE:
-                raise n_exc.Invalid(_('Expected active pool'))
+                raise n_exc.Invalid(_('Pool_id %(pool_id)s status ACTIVE '
+                                      'is expected but status is %(status)s') %
+                                    {'pool_id': pool_id,
+                                     'status': pool.status})
 
             retval = {}
             retval['pool'] = self.plugin._make_pool_dict(pool)