From 7f5749d5348d433d27b31fa772c1a49a819dd40e Mon Sep 17 00:00:00 2001 From: ZhaoBo Date: Tue, 8 Aug 2017 21:05:23 +0800 Subject: [PATCH] Fix sg_rule didn't set protocol field If the lb security group contains 1 rule without protocol, worker may raise error in _update_security_group_rules when listener creation. Change-Id: Idc826d251296435119ae963c832de29160062967 --- octavia/network/drivers/neutron/allowed_address_pairs.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/octavia/network/drivers/neutron/allowed_address_pairs.py b/octavia/network/drivers/neutron/allowed_address_pairs.py index e9263fd858..3ade93b333 100644 --- a/octavia/network/drivers/neutron/allowed_address_pairs.py +++ b/octavia/network/drivers/neutron/allowed_address_pairs.py @@ -152,7 +152,7 @@ class AllowedAddressPairsDriver(neutron_base.BaseNeutronDriver): # with the egress rules. VRRP uses protocol # 51 and 112 if rule.get('direction') != 'egress' and - rule.get('protocol').lower() == 'tcp'] + rule.get('protocol', '').lower() == 'tcp'] add_ports = set(updated_ports) - set(old_ports) del_ports = set(old_ports) - set(updated_ports) for rule in rules.get('security_group_rules', []):