X Tutup
Skip to content

Neutron: Add Description field to SecurityRules#167

Merged
olivergondza merged 1 commit intoopenstack4j:masterfrom
fapo85:master
Oct 31, 2021
Merged

Neutron: Add Description field to SecurityRules#167
olivergondza merged 1 commit intoopenstack4j:masterfrom
fapo85:master

Conversation

@fapo85
Copy link
Copy Markdown
Contributor

@fapo85 fapo85 commented Oct 29, 2021

PR description

as it was already indicated in an issue in the old repo, the description field is missing in the security rules.
ContainX/openstack4j#1290
with this PR i am submitting this feature/function.

Api doc: https://docs.openstack.org/api-ref/network/v2/index.html?expanded=show-security-group-rule-detail

Submitter checklist

Make sure that following is addressed to make the PR easier to process:

  • If applicable, changes are covered by either a unit or a functional test.
  • If applicable, changes ware verified manually against an OpenStack instance.
  • If the change is API related, the PR description links to OpenStack API documentation of that particular endpoint/request (https://docs.openstack.org/api-quick-start/#current-api-versions).
  • If the change concerns particular OpenStack service, its name is used as a PR name prefix (ex.: "Neutron: Add floatingIp port forwardings service").
  • If the PR closes existing GitHub issue, PR name have prefix: Fix #NNN: .

@olivergondza olivergondza merged commit a307ebf into openstack4j:master Oct 31, 2021
@olivergondza
Copy link
Copy Markdown
Member

Merging, thanks!

@fapo85
Copy link
Copy Markdown
Contributor Author

fapo85 commented Jun 14, 2022

@olivergondza When can I expect this to end up in a release?

@bitstorm
Copy link
Copy Markdown

@olivergondza When can I expect this to end up in a release?

it's available with the last 3.11

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.

3 participants

X Tutup