x/vmware-nsx
Code Issues Proposed changes

NSX|V3: Fix operator on LB advertisment rule

Browse Source 
For CIDR match we need it to be GE rather than EQ

Change-Id: I87b0bd097e1bfe4b2570ac8b63e398820e53ff3c
This commit is contained in:
Adit Sarfaty 2019-01-17 13:29:18 +02:00
parent de2302caae
commit 7b713575c9
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
vmware_nsx/services/lbaas/nsx_v3/implementation/lb_utils.py
View File

@ -236,7 +236,7 @@ def update_router_lb_vip_advertisement(context, core_plugin, router,
'display_name': ADV_RULE_NAME,
'action': nsx_constants.FW_ACTION_ALLOW,
'networks': external_cidrs,
'rule_filter': {'prefix_operator': 'EQ',
'rule_filter': {'prefix_operator': 'GE',
'match_route_types': ['T1_LB_VIP']}}
core_plugin.nsxlib.logical_router.update_advertisement_rules(
nsx_router_id, [adv_rule], name_prefix=ADV_RULE_NAME)

2
vmware_nsx/tests/unit/services/lbaas/test_nsxv3_driver.py
View File

@ -1011,7 +1011,7 @@ class TestEdgeLbaasV2Member(BaseTestEdgeLbaasV2):
[{'networks': [ext_cidr],
'display_name': lb_utils.ADV_RULE_NAME,
'rule_filter': {'match_route_types': ['T1_LB_VIP'],
'prefix_operator': 'EQ'},
'prefix_operator': 'GE'},
'action': 'ALLOW'}],
name_prefix=lb_utils.ADV_RULE_NAME)
mock_successful_completion = (