openstack/neutron-lbaas
Code Issues Proposed changes

Fix L7 Policy creation with Octavia driver

Browse Source 
The L7 Policy create method is sending the listener_id to Octavia
as a POST parameter, when the Octavia is not expecting it. This is
breaking L7 Policy creation with Octavia.

Change-Id: I8fabeee79cfc70da09b50cd2da6ec16f2742d824
Closes-Bug: #1607758
This commit is contained in:
Stephen Balukoff 2016-08-24 17:37:23 -07:00
parent b0741bf319
commit 15e6936499
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
neutron_lbaas/drivers/octavia/driver.py
View File

@ -510,8 +510,6 @@ class L7PolicyManager(driver_base.BaseL7PolicyManager):
args['id'] = l7p.id args['id'] = l7p.id
if not graph: if not graph:
if l7p.listener_id:
args['listener_id'] = l7p.listener_id
return args return args
if (l7p.redirect_pool and l7p.action == if (l7p.redirect_pool and l7p.action ==

2
neutron_lbaas/tests/unit/drivers/octavia/test_octavia_driver.py
View File

@ -73,7 +73,7 @@ class BaseOctaviaDriverTest(test_db_loadbalancerv2.LbaasPluginDbTestCase):
hm = data_models.HealthMonitor(id=id, pool=pool) hm = data_models.HealthMonitor(id=id, pool=pool)
sp = data_models.SessionPersistence(pool_id=pool.id, pool=pool) sp = data_models.SessionPersistence(pool_id=pool.id, pool=pool)
l7policy = data_models.L7Policy( l7policy = data_models.L7Policy(
id=id, listener=listener, id=id, listener=listener, listener_id=listener.id,
action=constants.L7_POLICY_ACTION_REDIRECT_TO_POOL) action=constants.L7_POLICY_ACTION_REDIRECT_TO_POOL)
l7rule = data_models.L7Rule( l7rule = data_models.L7Rule(
id=id, policy=l7policy, type=constants.L7_RULE_TYPE_PATH, id=id, policy=l7policy, type=constants.L7_RULE_TYPE_PATH,