Skip to content

[v1] networking/v2/ports: allow list filter by security group#2729

Merged
EmilienM merged 1 commit intov1from
bp-v1-e767be1
Aug 25, 2023
Merged

[v1] networking/v2/ports: allow list filter by security group#2729
EmilienM merged 1 commit intov1from
bp-v1-e767be1

Conversation

@gophercloud-backport-bot
Copy link
Copy Markdown

Backport: #2728

neutron v2 ports APIs allow to list ports by security group already:
https://docs.openstack.org/api-ref/network/v2/#show-port-details

This patch adds the SecurityGroups field to ListOpts.

One way to filter the ports by security group can be done with the
following code:

listOpts := ports.ListOpts{
	SecurityGroups: []string{"2183457b-70cc-4fd0-a2dc-95323fa19e45"}
}

allPages, err := ports.List(networkClient, listOpts).AllPages()
if err != nil {
	panic(err)
}

allPorts, err := ports.ExtractPorts(allPages)
if err != nil {
	panic(err)
}

for _, port := range allPorts {
	fmt.Printf("%+v
", port)
}

neutron v2 ports APIs allow to list ports by security group already:
https://docs.openstack.org/api-ref/network/v2/#show-port-details

This patch adds the `SecurityGroups` field to `ListOpts`.

One way to filter the ports by security group can be done with the
following code:

```
listOpts := ports.ListOpts{
	SecurityGroups: []string{"2183457b-70cc-4fd0-a2dc-95323fa19e45"}
}

allPages, err := ports.List(networkClient, listOpts).AllPages()
if err != nil {
	panic(err)
}

allPorts, err := ports.ExtractPorts(allPages)
if err != nil {
	panic(err)
}

for _, port := range allPorts {
	fmt.Printf("%+v\n", port)
}
```
@coveralls
Copy link
Copy Markdown

coveralls commented Aug 25, 2023

Coverage Status

coverage: 79.122%. remained the same when pulling 45e856e on bp-v1-e767be1 into c6bdb0f on v1.

@EmilienM EmilienM added the semver:minor Backwards-compatible change label Aug 25, 2023
@EmilienM EmilienM merged commit 51b35b4 into v1 Aug 25, 2023
@EmilienM EmilienM deleted the bp-v1-e767be1 branch August 25, 2023 13:53
@pierreprinetti pierreprinetti added the v1 This PR targets v1 label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:minor Backwards-compatible change v1 This PR targets v1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants