x/vmware-nsx
Code Issues Proposed changes

Fix LBaaS Haproxy occurs error if no member is added

Browse Source 
If no member is added and session_persistence.type=HTTP_COOKIE,
haproxy agent would not add cookie persistence option to the backend.
Closes-Bug: #1302283

Change-Id: Ifa2564df924c2555225a749a99c705b3f1caab4a
This commit is contained in:
berlin 2014-04-11 08:04:47 +08:00
parent a701f23807
commit f1463640ba
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
neutron/services/loadbalancer/drivers/haproxy/cfg.py
View File

@ -199,7 +199,8 @@ def _get_session_persistence(config):
if persistence['type'] == constants.SESSION_PERSISTENCE_SOURCE_IP: if persistence['type'] == constants.SESSION_PERSISTENCE_SOURCE_IP:
opts.append('stick-table type ip size 10k') opts.append('stick-table type ip size 10k')
opts.append('stick on src') opts.append('stick on src')
elif persistence['type'] == constants.SESSION_PERSISTENCE_HTTP_COOKIE: elif (persistence['type'] == constants.SESSION_PERSISTENCE_HTTP_COOKIE and
config.get('members')):
opts.append('cookie SRV insert indirect nocache') opts.append('cookie SRV insert indirect nocache')
elif (persistence['type'] == constants.SESSION_PERSISTENCE_APP_COOKIE and elif (persistence['type'] == constants.SESSION_PERSISTENCE_APP_COOKIE and
persistence.get('cookie_name')): persistence.get('cookie_name')):

8
neutron/tests/unit/services/loadbalancer/drivers/haproxy/test_cfg.py
View File

@ -178,7 +178,15 @@ class TestHaproxyCfg(base.BaseTestCase):
self.assertEqual(cfg._get_session_persistence(config), self.assertEqual(cfg._get_session_persistence(config),
['stick-table type ip size 10k', 'stick on src']) ['stick-table type ip size 10k', 'stick on src'])
config = {'vip': {'session_persistence': {'type': 'HTTP_COOKIE'}},
'members': []}
self.assertEqual([], cfg._get_session_persistence(config))
config = {'vip': {'session_persistence': {'type': 'HTTP_COOKIE'}}} config = {'vip': {'session_persistence': {'type': 'HTTP_COOKIE'}}}
self.assertEqual([], cfg._get_session_persistence(config))
config = {'vip': {'session_persistence': {'type': 'HTTP_COOKIE'}},
'members': [{'id': 'member1_id'}]}
self.assertEqual(cfg._get_session_persistence(config), self.assertEqual(cfg._get_session_persistence(config),
['cookie SRV insert indirect nocache']) ['cookie SRV insert indirect nocache'])