Remove filtering by device_owner. (#2304)

gryf · web-flow · commit a2c86f95b652 · 2023-07-18T01:03:10.000-07:00
In OpenStack default AZ is called 'nova', thus device_owner is filled
with 'compute:nova', and that's ok for deployments within one (default)
AZ.

Filtering by device_owner however, will make function getAttachedPorts
to return zero ports for provided node. In this patch, filtering by
device_owner is simply removed.
diff --git a/pkg/openstack/instances.go b/pkg/openstack/instances.go
@@ -703,8 +703,7 @@ func getAddressesByName(client *gophercloud.ServiceClient, name types.NodeName,
 // getAttachedPorts returns a list of ports attached to a server.
 func getAttachedPorts(client *gophercloud.ServiceClient, serverID string) ([]ports.Port, error) {
 	listOpts := ports.ListOpts{
-		DeviceID:    serverID,
-		DeviceOwner: "compute:nova",
+		DeviceID: serverID,
 	}
 
 	return openstack.GetPorts(client, listOpts)

Original file line number	Diff line number	Diff line change
`@@ -703,8 +703,7 @@ func getAddressesByName(client *gophercloud.ServiceClient, name types.NodeName,`
`703`	`703`	`// getAttachedPorts returns a list of ports attached to a server.`
`704`	`704`	`func getAttachedPorts(client *gophercloud.ServiceClient, serverID string) ([]ports.Port, error) {`
`705`	`705`	`listOpts := ports.ListOpts{`
`706`		`- DeviceID: serverID,`
`707`		`- DeviceOwner: "compute:nova",`
	`706`	`+ DeviceID: serverID,`
`708`	`707`	`}`
`709`	`708`
`710`	`709`	`return openstack.GetPorts(client, listOpts)`