Skip to content

Commit f7c0850

Browse files
committed
[FWaaS] Remove client side protocol validation
There is no need to limit available choices for the firewall rule's protocol on the client side. Neutron-fwaas plugin on the server side will do the validation in the same way as for security group rules. And for SG rules OSC is not validating nor limiting choices on the client's side at all. Closes-bug: #2142479 Change-Id: I8c02a2232601c2ab6655c458aa0365102b3b5e2d Signed-off-by: Slawek Kaplonski <skaplons@redhat.com>
1 parent d9a3518 commit f7c0850

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

neutronclient/osc/v2/fwaas/firewallrule.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,9 +86,12 @@ def _get_common_parser(parser):
8686
help=_('Description of the firewall rule'))
8787
parser.add_argument(
8888
'--protocol',
89-
choices=['tcp', 'udp', 'icmp', 'any'],
9089
type=nc_utils.convert_to_lowercase,
91-
help=_('Protocol for the firewall rule'))
90+
help=_('IP protocol (ah, dccp, egp, esp, gre, icmp, igmp, '
91+
'ipv6-encap, ipv6-frag, ipv6-icmp, ipv6-nonxt, ipv6-opts, '
92+
'ipv6-route, ospf, pgm, rsvp, sctp, tcp, udp, udplite, '
93+
'vrrp and integer representations [0-255] or any; '
94+
'default: any (all protocols))'))
9295
parser.add_argument(
9396
'--action',
9497
choices=['allow', 'deny', 'reject'],

0 commit comments

Comments
 (0)