Skip to content

Add Missing Neutron v2 Protocol Constant "any"#2310

Merged
EmilienM merged 2 commits intogophercloud:masterfrom
HappyFX:master
Jan 9, 2022
Merged

Add Missing Neutron v2 Protocol Constant "any"#2310
EmilienM merged 2 commits intogophercloud:masterfrom
HappyFX:master

Conversation

@HappyFX
Copy link
Copy Markdown
Contributor

@HappyFX HappyFX commented Jan 7, 2022

Add option "Any" in security group rules, according the documentation:

Network version 2:
IP protocol (ah, dccp, egp, esp, gre, icmp, igmp, ipv6-encap, ipv6-frag, ipv6-icmp, ipv6-nonxt, ipv6-opts, ipv6-route, ospf, pgm, rsvp, sctp, tcp, udp, udplite, vrrp and integer representations [0-255] or any; default: any (all protocols))

Fixes #2309

In code neutron None - wildcard, but in documentation it mention as any
Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

openstack/python-openstackclient@82f45d9
https://github.com/openstack/neutron/blob/1ad9ca56b07ffdc9f7e0bc6a62af61961b9128eb/neutron/extensions/securitygroup.py#L211
https://github.com/openstack/neutron/blob/1ad9ca56b07ffdc9f7e0bc6a62af61961b9128eb/neutron/extensions/securitygroup.py#L151

Add option "Any" in security group rules, according the [documentaion](https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/security-group-rule.html#cmdoption-openstack-security-group-rule-create-protocol):

```bash
Network version 2:
IP protocol (ah, dccp, egp, esp, gre, icmp, igmp, ipv6-encap, ipv6-frag, ipv6-icmp, ipv6-nonxt, ipv6-opts, ipv6-route, ospf, pgm, rsvp, sctp, tcp, udp, udplite, vrrp and integer representations [0-255] or any; default: any (all protocols))
```
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jan 7, 2022

Build failed.

@HappyFX
Copy link
Copy Markdown
Contributor Author

HappyFX commented Jan 7, 2022

I believe the broken test due QoS neutron changes: openstack/neutron@cf8fa0d

@coveralls
Copy link
Copy Markdown

coveralls commented Jan 8, 2022

Coverage Status

Coverage remained the same at 79.894% when pulling 7d3e617 on HappyFX:master into 6626c35 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jan 8, 2022

Build failed.

@EmilienM EmilienM merged commit cb4f6f2 into gophercloud:master Jan 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add options any

3 participants